Les fichiers composer.json et composer.lock doivent être commités en même temps

Beaucoup de modification sur le fichier composer.json sont commitées sans que le nouveau fichier composer.lock soit inclus. Dans ce cas les modification du composer.json ne sont pas inclus par la commande composer install et cela pose des problèmes pour le débug.

Règle

Dans un commit, si le le fichier composer.json est modifié, alors le fichier composer.lock doit aussi être modifié et inclus dans ce commit.

à faire

• améliorer la documentation (menmento, ...) pour expliquer --> !352 (merged)
• imposer un hook git (via composer) pour forcer cette vérification à chaque commit --> !344 (merged)
• ajouter ce même test à la CI ---> via la commande composer validate --> !341 (merged)

Exemple

git log --pretty="%C(Yellow)%h  %C(reset)%ad (%C(Green)%cr%C(reset))%x09 %C(Cyan)%an: %C(reset)%s" composer.json
git log --pretty="%C(Yellow)%h  %C(reset)%ad (%C(Green)%cr%C(reset))%x09 %C(Cyan)%an: %C(reset)%s" composer.lock

composer.json

composer.lock