Revue de code

Pour mettre en commun nos compétences et nos codes, et nous permettre de constituer une équipe "tech" à part entière, nous mettons en place un système de revue de code.

Les revues de code sont mis en place de manière systématique sur toutes les parties du code qui concernent tout ou partie de l'équipe. Cela inclut les dépôts :

• sder (https://github.com/Cour-de-cassation/sder)
• nlp-jurizonage-api (https://github.com/Cour-de-cassation/nlp-jurizonage-api)
• nlp-pseudonymisation-api (https://github.com/Cour-de-cassation/nlp-pseudonymisation-api)
• openjustice-sder (https://github.com/Cour-de-cassation/openjustice-sder).

Mise en place et déroulé d'une revue de code :

• Prérequis : le travail sera effectué sur des branches, on évitera au maximum les commits directement sur master (sauf nécessité), on préférera passer par une Pull Request. La branche master pourra le cas échéant être protégée en écriture.
• Pour chaque Pull Request, on assigne des correcteurs. Comme l'équipe est petite, nous pouvons assigner systématiquement tous les autres membres de l'équipe à chaque Pull Request. On veillera à rendre la Pull Request auto suffisante pour les correcteurs.
• Le correcteur s'engage à répondre sous un jour ouvré (pour cela, il ne faut pas oublier d'activer ses notifications Github). Si un correcteur a oublié de répondre, il vaut mieux le relancer que de passer outre son avis. En cas de congés, il n'est pas nécessaire d'attendre. Il est possible de passer outre la revue de code (en terme de temps ou en terme d'avis) si le contexte de développement l'exige.