DevCamp presentation: PR to merge or not to merge: Difference between revisions

DevCamp presentation: PR to merge or not to merge (view source)

402 bytes added , 2 years ago

m

30,909

edits

@@ Line 25: / Line 25: @@
 [[File:Usecase2.png|center|thumb]]
-*sur la branche « develop » on n’ajoute pas de variable globale et l'on priorise la qualité de code
+*L'usage d'un variable globale (qui normalement doit être évité) est acceptable pour un fix sur une branche de maintenance car il permet une correction du bug "à moindre impact". Mais sur la branche « develop », l'ajout de variable globale n'est pas toléré car on priorise la qualité de code long terme. La solution est donc le compromis suivant:
-**ici le mergeur va saisir TODO ou Fix me ou créer une Issue pour rappeler qu'il faudra revoir le code pour la branche "develop"
+**ici le mergeur va accepter la PR sur la branche ancienne pour obtenir la stabilisation de la vieille version en maintenance, mais va saisir "@TODO" ou "@FIXME" dans le code, ou créer une Issue pour rappeler qu'il faudra revoir ce code pour la branche "develop".
-*le développeur aurait du faire une version différente sur la develop pour ne pas utiliser la variable globale
+*le développeur est invité à faire une version différente sur la branche "develop" pour ne pas utiliser la variable globale.
 '''Cas 3'''