Vérifier la syntaxe PHP à chaque commit

Afin d’automatiser et d’améliorer la qualité du code, il est possible de lancer certaines tâches lors de chaque commit avec Git. Des outils comme GrumPHP permettent de faire cela mais il peut arriver qu’il soit incompatible avec votre projet PHP (et puis c’est toujours intéressant de savoir le faire à la main).

Les hooks

Les hooks sont des petits scripts qui se lancent lors des différentes actions Git. Il en existe plusieurs et pour les connaitre il suffit de se rendre dans un dépot Git et de faire:

ls .git/hooks/
applypatch-msg.sample  pre-applypatch.sample          pre-commit.sample          pre-push.sample    update.sample
commit-msg.sample      post-update.sample             prepare-commit-msg.sample  pre-rebase.sample

Les fichiers .sample sont des examples proposés par Git. Dans notre cas, on va partir de zéro en créant un script vide

touch .git/hooks/pre-commit

Le script

Nous allons donc commencer par afficher les fichiers contenu dans le commit. Pour cela on s’appuie sur la commande git diff --cached --name-only qui affiche les uniquement le nom des fichiers modifié. On rajoute un petit grep par dessus pour filtrer uniquement les fichiers PHP et on boucle dessus.

Voici le résultat:

#!/bin/bash
# .git/hooks/pre-commit

# loop on all commited PHP files
for file in $(git diff --cached --name-only | grep -E '.php$') ; do
  echo "[x] $file"
done

Il faut ensuite faire le tri dans ses fichiers car Git nous remonte aussi ceux supprimés, il faut donc rajouter une condition:

#!/bin/bash
# .git/hooks/pre-commit

# loop on all commited PHP files
for file in $(git diff --cached --name-only | grep -E '.php$') ; do
  # first we verify that file exists
  if [ -f $file ]; then
    echo "[x] $file"
  fi
done

Et pour terminer on vérifie la syntaxe à l’aide de la commande php -l "$file" et si on trouve une erreur, on stoppe l’execution avec un signal d’erreur à l’aide de la fonction exit 1.

#!/bin/bash
# .git/hooks/pre-commit

# loop on all commited PHP files
for file in $(git diff --cached --name-only | grep -E '.php$') ; do
  # first we verify that file exists
  if [ -f $file ]; then
    # check syntax & catch errors
    if ($( php -l "$file" 1> /dev/null  )); then
      echo "[x] $file"
    else
      # file contains syntax error, so we'll stop programm & cancel commit
      echo "[ ] $file"
      exit 1
    fi
  fi
done

Le test

Notre script étant prêt, il nous reste plus qu’à tester. On ajoute un fichier PHP valide et on vérifie qu’il est accepté

echo '<?php echo "test";' > valid.php
git add valid.php
git commit -m "Add valid file"

Maintenant on teste avec un fichier PHP non valide

git reset --hard HEAD^ # on annule le précedent commit
echo '<?php echo "test;' > invalid.php
git add invalid.php
git commit -m "Add invalid file"

Le script coupe l’ajout du commit!

Axes d’améliorations

Nous avons vu à travers un exemple simple qu’il est possible d’automatiser les tâches les pluscourant. On pourrait aller plus loin et:

• Vérifier la syntaxe des fichier Ruby avec la commande ruby -c file
• Formatter automatiquement les fichier commité
• Lancer les test unitaires

Voir les autres articles liés