|
|
# Sommaire
|
|
|
1. [Commandes Git](/Git#commandes-git)
|
|
|
2. [Review](/Git#review)
|
|
|
|
|
|
# Commandes Git
|
|
|
|
|
|
## 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 apporter par le relecteur
|
|
|
* Traitement des retours en signalant dans un commentaire associé a l'erreur "corrigé"
|
|
|
|
|
|
Le relecteur procède ensuite a une vérification des correction et valide celle-ci en marquant le thread comme résolu.
|
|
|
![image](uploads/3cda48c64a8e14acfa2414a87856ff8f/image.png)
|
|
|
|
|
|
Une fois tous les problèmes résolu, la fonctionnalité peut être merge dans la branche _dev_.
|
|
|
|
|
|
## Exemple
|
|
|
![image](uploads/7dfe3cf68c8dc141bbee3be09ddfbcf4/image.png) |
|
|
\ No newline at end of file |