Commandes
Merge d'une branche vers le dev (ou vers une autre branche)
S'assurer que la branche est bien up-to-date
git checkout <branch_name>
git pull
git commit (si besoin)
S'assurer que le dev est bien up-to-date
git checkout dev
git pull dev
git commit (si besoin)
Merge de dev vers la branche
git checkout <branch_name>
git merge dev
git commit (si besoin)
git push (si besoin)
Même procédure depuis dev
git checkout dev
git merge <branch_name>
git push (normalement pas de conflit si pas de commit depuis le début de la procédure)
Création d'une nouvelle branche
Création de la nouvelle branche en local à partir de la branche courante
git checkout -b <branch_name>
Push de la nouvelle branche sur le repo (-u pour set-upstream)
git push -u origin <branch_name>
Supprimer une branche
Suppression de la branche locale en vérifiant qu'elle est bien à jour avec la remote
git branch -d <branch_name>
Suppression (forcée) de la branche locale sans vérifier si elle est à jour avec la remote
git branch -D <branch_name>
Suppression de la branche remote
git push origin --delete <branch_name>
Review
Fonctionnement
Tous les membres de l'équipe effectuent des relectures croisées sur les différentes US réalisées lors d'un sprint. Les relectures sont tournantes afin de varier le relecteur.
Flow
Une US terminée par un développeur doit faire l'objet d'une merge request sur la forge. Le développeur en charge de la review doit ensuite se charger des tâches suivantes :
- Vérification que le développement est en accord avec la ou les maquettes
- Relecture du code
- Ajout de commentaires sur les éventuels problèmes ou incompréhensions
- Correction des erreurs minimes afin de faciliter le travail du développeur
- Ajout d'un pouce sur la merge request afin de signaler que celle-ci a été vérifiée.
Le développeur responsable de la tâche doit a son tour s’acquitter des tâches suivantes:
- Lecture des commentaires
- Compréhension des modifications minimes apportées par le relecteur
- Traitement des retours en signalant dans un commentaire associé a l'erreur "corrigée"
Le relecteur procède ensuite a une vérification des corrections et valide celles-ci en marquant le thread comme résolu.
Une fois tous les problèmes résolus, la fonctionnalité peut être merge dans la branche dev.