Vous en avez assez, quand vous relisez le code d’une pull request, de passer trop de temps sur des détails (importants, mais chronophages) comme :
- Le code est-il conforme aux PSR ?
- Le développeur a-t-il pensé à ajouter la clé de config dans le bon fichier ?
- Les migrations SQL ont-elles été générées ?
- …
- Toutes ces règles propres à vos applications qu’il faut sans cesse rappeler aux développeurs ?
Voilà le genre de situation que nous essayons d’éviter :
Bonne nouvelle : le bot PR Review Watcher est fait pour vous.
Dès qu’une nouvelle pull request est créée, il publie en commentaire la liste des vérifications que vous avez définies. Il suffit d’installer le projet via composer, puis de configurer dans l’interface la liste des vérifications à publier sur vos dépôts. Vous pouvez même définir des vérifications différentes pour chaque branche.
Votre équipe gagne ainsi du temps : plus besoin de traquer avec insistance les erreurs courantes, elle peut se concentrer sur l’architecture du code, les algorithmes, la compréhension…
N’hésitez pas à contribuer et à envoyer des PR (le projet est écrit avec Silex).
PS : le style de code n’est qu’un exemple pour illustrer, de façon générale, l’intérêt du projet. Entre nous, dans mon équipe, un hook de pre-commit empêche les développeurs de commiter du code non conforme à PSR2.