[1.13] mise à jour composer.lock

[wouldsmina]

Je me suis un peu planté sur l'upgrade d'une lib composer, il ne suffit pas de modifier le fichier à la main...
bizarrement j'avais systématiquement la bonne version sur mon poste... Un effet du cache de composer peut être.

P.S. : Bernard et TxR s'occupe du test. @prytoegrian, si tu as un peu de temps, jette un coup d’œil au code stp

[prytoegrian]

Je suis pas sûr de comprendre, que s'est il passé?

[wouldsmina]

bah je pensais qu'il suffisait de faire ça pour mettre à jour une lib composer, mais ça ne semble pas faire le job. L'API reste en 1.7.0. J'ai vu dans composer.lock un certain nombre de paramètre pour libertempo/api dont une url qui pointait sur la 1.7.0. J'ai donc fait un composer update --lock ce qui a eu pour effet de mettre à jour composer.lock avec cette fois une url qui pointe bien vers l'API 1.9.0.
Ça a aussi eu pour effet la modification du fichier package-lock.json. N'étant pas encore bien à l'aise avec composer je ne saurai dire si c'est normal ou pas. En tout cas, sur une vm clean j'ai bien l'API 1.9.0.

[prytoegrian]

Ok, je regarde ce soir. C'est tout bête tu verras 😉

[prytoegrian]

Ah oui mais si j'oublie de poster aussi… En effet, il ne suffit pas de modifier le composer.json car quand tu fais composer install, il vérifie en premier le composer.lock. Aussi, quand tu veux mettre à jour, il te faut faire une PR où tu changes la version de ton fichier, puis tu fais un composer update afin de ne pas tenir compte du .lock. Ce dernier va se mettre à jour et tu pourras commit ces deux fichiers pour que j'en profite de mon côté. Un peu comme là : #855 🥇
C'est pareil pour le js.

[wouldsmina]

bernard et TxR ont toujours la 1.7.0 :/

[prytoegrian]

... Ok, je vérifie.

[prytoegrian]

Changer ces fichiers n'a (presque) pas d'impact en l'état. Si tu veux qu'ils soient pleinement utilisés, alors tu dois dire à composer de charger les dernières versions des logiciels.
Nous avons une commande make update qui fait ça sans avoir d'autres impacts :-)

[wouldsmina]

pour ma part, j'ai bien la 1.9.0 en faisant un make update, mais pas Bernard et TxR. Il y a quelques choses qui nous échappent...

[prytoegrian]

Ils sont bien sur la branche relative à la PR où on est là ?

[wouldsmina]

à priori c'est quelque chose dans le genre. Un nouvel essai en partant de zéro donne le résultat attendu. BPM! :)