30,909
edits
DevCamp presentation: PR to merge or not to merge: Difference between revisions
Jump to navigation
Jump to search
m
DevCamp presentation: PR to merge or not to merge (view source)
Revision as of 09:49, 10 June 2024
, 2 years ago→Quelques exemples
Tag: 2017 source edit |
Tag: 2017 source edit |
||
| Line 26: | Line 26: | ||
*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: | *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 accepter la PR sur la branche ancienne pour obtenir la stabilisation de la vieille version en maintenance, mais va | **ici le mergeur va accepter la PR sur la branche ancienne pour obtenir la stabilisation de la vieille version en maintenance, mais va ajouter une ligne de commentaire incluant "@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 est invité à faire une version différente sur la branche "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. | ||