-
Notifications
You must be signed in to change notification settings - Fork 0
Pull Requests & Merge
ATTENTION: Cette page utilise les connaissances des pages:
Dans cette section, vous pouvez retrouver les références pour les topics des sections suivantes:
Ces protocoles sont strict et rigide afin de préserver l'intégritée du code et des fonctionnalitées mises au service des utilisateurs et/ou testeurs.
Pour pouvoir merge du code vers une branche, il est obligatoire d'utiliser le système de PR (Pull Request) afin de pouvoir échanger à propos du code à merge.
Plusieures règles viennent s'appliquer à celà:
- La personne ayant créer la PR ne peut approuver son propre code.
- Toute PR doit être validée et review par un ou des membre(s) de l'EIP étant dans votre pôle.
- La personne ayant créer la PR ne peut lancer le merge vers la branche de destination.
Ces règles sont là pour maintenir la qualitée du code ainsi que la compréhension des diffiultées rencontrées durant le développement.
ATTENTION: Une branche de Feature ne peut pas à être directement faire l'objet d'une PR ou d'un merge directement vers PROD, dans le cas contraire les commits seront supprimés.
Dans cette section, vous pouvez retrouver les références pour les topics des sections suivantes:
Une fois une feature finie, la branche est prête à être soumise à une PR vers la branche DEV.
Pour pouvoir merge une PR doit être crée.
Une Feature doit venir avec des tests unitaire testant la Feature, et la Feature doit pourvoir passer les tests unitaire écrits.
Une fois la PR faite, il y a 2 options:
- La PR peut être Approuvée
- La PR peut demander des changements
Dans la première situtation la PR sera donc merge par un membre de votre pôle comme expliqué précédement.
Dans le second cas, vous êtes libre de débattre des changements avec vos reviewer (de grande préférence sur la PR afin de garder une trace écrite).
Une fois une PR faite, il faut review cette dernière.
La vérification d'une PR doit passer par plusieures étapes:
- Vérification de la compréhension et de la qualitée du code
- Vérification de la validitée des tests unitaire
- Vérification que la Feature passe les tests unitaires
- Vérification de l'accord de l'objectif de la Feature et de son résultat
Dans le cas où toute les vérifications sont validées la PR, elle peut être approuvée.
Dans le cas où il y a un ou des obstacles à la validation la PR doit faire l'objet d'une review expliquant précisement le refus afin que le developpeur puisse ajuster son code.
Une fois une version finie et prête à être déployée, la branche est prête à être soumise à une PR vers la branche PROD.
La PR peut être crée par n'importe qui après approbation que la branche DEV représente une version déployable.
Toute PR vers PROD doit être accompagné de tests d'intégrations ou ajout à ses tests.
Une fois la PR faite, tous les membres du groupe doivent review la PR de la branche DEV.
Voici les vérification d'une PR destinée à être merge vers PROD:
- Vérification de la compréhension et de la qualitée du code
- Vérification de la validitée des tests unitaire sur les Features ajoutées
- Vérification que les Features passe les tests unitaires
- Vérification de l'intégritée du projet et de ses Features précédentes
Chaque membre doit approuver la PR afin d'être merge vers PROD.
Tout obstacle au déploiement du merge doit faire l'objet d'une review détaillée.
Une fois toutes ces tâches validées la PR peut être merge ver PROD.