Open brulej opened 4 years ago
oliviercailloux
commented
4 years ago Les instructions indiquent, je pense : si votre PR dépend d’une autre, il faut attendre la fusion de l’autre ; si elle n’en dépend pas, vous pouvez l’ouvrir sans que ça pose de problème de visibilité des changements.
Dans quel catégorie se situe votre cas ? Si les instructions ne conviennent pas, pourriez-vous m’indiquer pourquoi ?
Raphda
commented
4 years ago Bonjour Monsieur,
Notre cas se situe dans cette situation :
Une tâche est effectuée en une seule itération. Une fois notée, cette tâche doit être considérée comme définitivement terminée (cela permet de s’y retrouver dans les notes). Si votre tâche poursuit une tâche d’une livraison précédente, il s’agit quand-même d’une nouvelle tâche. Créez une nouvelle branche (par exemple « unit-tests-Truc-it-3 ») partant de votre branche existante. Lorsque vous créez la nouvelle PR, indiquez en commentaire que la tâche fait suite à la PR #x. (FOLLOW-UP)
Cependant, faut-il faire une PR sur de la branche master, ou sur la branche de l'itération précédente ?
oliviercailloux
commented
4 years ago Cela ne m’indique pas si votre tâche dépend d’une autre PR ou pas.
Pour répondre à votre dernière question, il faut toujours faire des PR vers la branche master, ce qui me permet de reviewer tout les changements proposés (comme si on continuait sur la même PR). Mais je soupçonne que ça ne répond pas à votre interrogation (que je crains ne pas comprendre).
camillelanglois3
commented
4 years ago Bonjour Monsieur,
Nous souhaitons vous faire une PR contenant les modifications que vous avez demandées suite à la PR #26. Cependant, comme nous en sommes à l'itération suivante, nous avons cru comprendre dans les instructions qu'il fallait fermer la PR #26 et en créer une nouvelle indiquant qu'elle dépendait de la #26. Nous avons également pu comprendre que vous ne souhaitiez pas revoir les modifications que vous aviez déjà reviewé auparavant, mais en faisant cette nouvelle PR vous verrez également toutes les modifications que nous avions effectuées lors de l'itération précédente, puisque la PR #26 n'a pas été validée.
Que devons nous faire ?
Bonjour @oliviercailloux,
L'équipe SVG souhaite ouvrir une PR pour l'it4 java mais cela soulève des problèmes de visibilité des modifications effectuées. Comme la PR #26 n'a pas pu être merge avec la branche master, nous nous retrouvons face à 2 solutions imparfaites:
1- Merge avec master. Dans ce cas votre review, va être parasitée par le travail réalisé durant l'it3 (6 fichiers modifiés) 2- Merge avec la branche de la Pr #26 . Dans ce cas votre review, vas être parasitée par le travail réalisé en UML (32 fichiers modifiés)
Pouvez-vous nous aider quant sur la marche à suivre ?
Bien cordialement,
Jérémie