mirror of
https://github.com/YunoHost/project-organization.git
synced 2024-09-03 19:56:35 +02:00
Le groupe peut fournir un template des infos pour décrire une PR
This commit is contained in:
parent
0859246009
commit
af42f2a000
1 changed files with 6 additions and 5 deletions
|
@ -102,11 +102,13 @@ Si un consensus ne peut être trouvé au sein d'un gorupe en suivant le processu
|
||||||
|
|
||||||
N'importe quel contributeur peut proposer une pull request (abrégée PR dans la suite) dans les divers dépôts liés au projet YunoHost (core, apps, infra, ...).
|
N'importe quel contributeur peut proposer une pull request (abrégée PR dans la suite) dans les divers dépôts liés au projet YunoHost (core, apps, infra, ...).
|
||||||
|
|
||||||
La proposition doit obligatoirement décrire les points suivants :
|
L'auteur est vivement encouragé à décrire sa proposition des informations
|
||||||
- problème auquel réponds la PR
|
pertinentes. Le groupe peut, à cette fin, proposer un modèle des informations à
|
||||||
- solution, stratégie, résumé des changements, et/ou choix techniques utilisés dans la PR
|
inclure, comme par exemple :
|
||||||
- comment tester la PR (sauf si s'il n'y a rien a tester)
|
|
||||||
- status actuel de la PR (ex. : non terminé, en attente de revues, choix techniques à faire...)
|
- status actuel de la PR (ex. : non terminé, en attente de revues, choix techniques à faire...)
|
||||||
|
- problème auquel réponds la PR (et références liées, par ex. : ticket sur le bugtracker, post sur le forum...)
|
||||||
|
- solution, stratégie, résumé des changements, et/ou choix techniques utilisés dans la PR
|
||||||
|
- comment tester la PR
|
||||||
|
|
||||||
L'auteur est vivement encouragé à respecter les bonnes pratiques suivantes :
|
L'auteur est vivement encouragé à respecter les bonnes pratiques suivantes :
|
||||||
- une PR doit concerner exclusivement un sujet précis. Par exemple, elle ne doit pas à la fois résoudre un bug et ajouter une fonctionnalité (à moins que l'un implique l'autre) ;
|
- une PR doit concerner exclusivement un sujet précis. Par exemple, elle ne doit pas à la fois résoudre un bug et ajouter une fonctionnalité (à moins que l'un implique l'autre) ;
|
||||||
|
@ -131,7 +133,6 @@ Une fois la PR déclarée comme terminée, les contributeurs sont invités à do
|
||||||
|
|
||||||
Les relecteurs rapportent également le degré de relecture et de tests effectués (c.f. liste ci-dessous). Selon l'importance de la PR (mineure, moyenne ou majeure), différents quotas de tests et approbations sont à remplir pour que celle-ci soit validée. Les relecteurs peuvent valider une fois chaque type de relecture/test nécessaire (par exemple, un relecteur peut donner un point d'accord sur le principe, un autre point de relecture en diagonale, et un autre point de test dans des cas simples.). L'auteur de la PR ne compte pas dans ces quotas de validation. La proposition doit aussi passer les tests automatiques disponibles dans le groupe (CI, tests unitaires/fonctionnels, linter, ...).
|
Les relecteurs rapportent également le degré de relecture et de tests effectués (c.f. liste ci-dessous). Selon l'importance de la PR (mineure, moyenne ou majeure), différents quotas de tests et approbations sont à remplir pour que celle-ci soit validée. Les relecteurs peuvent valider une fois chaque type de relecture/test nécessaire (par exemple, un relecteur peut donner un point d'accord sur le principe, un autre point de relecture en diagonale, et un autre point de test dans des cas simples.). L'auteur de la PR ne compte pas dans ces quotas de validation. La proposition doit aussi passer les tests automatiques disponibles dans le groupe (CI, tests unitaires/fonctionnels, linter, ...).
|
||||||
|
|
||||||
|
|
||||||
| | **Mineure** | **Moyenne** | **Majeure** |
|
| | **Mineure** | **Moyenne** | **Majeure** |
|
||||||
|-----------------------------------|-------------|--------------|-------------|
|
|-----------------------------------|-------------|--------------|-------------|
|
||||||
| **Accord sur le principe** | 2 | 3 | 4 |
|
| **Accord sur le principe** | 2 | 3 | 4 |
|
||||||
|
|
Loading…
Reference in a new issue