From 99aab8e10d014bdbb9b012eea2f2593b2b516c3a Mon Sep 17 00:00:00 2001 From: Alexandre Aubin Date: Mon, 10 Jul 2017 17:20:07 +0200 Subject: [PATCH] Tentative de clarification qu'un reviewer peut cocher plusieurs cases --- yunohost_project_organization_fr.md | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/yunohost_project_organization_fr.md b/yunohost_project_organization_fr.md index 16d0200..34fa93a 100644 --- a/yunohost_project_organization_fr.md +++ b/yunohost_project_organization_fr.md @@ -135,10 +135,10 @@ Il appartient aussi à l'auteur de la PR de juger de son importance. (Ce jugemen Une fois la PR déclarée comme terminée, les contributeurs sont invités à donner leurs avis, relire et tester les changements proposés pour les valider. Lorsque des bugs ou des implémentations mauvaises ou incomplètes sont trouvées, les relecteurs rapportent cordialement le problème à l'auteur de la PR sur le fil de discussion. Si le problème trouvé est simple à corriger (e.g. typo ou détail de forme), le relecteur est encouragé à amender la PR pour corriger le problème lui-même. Sinon, l'auteur fait de son mieux pour corriger les problèmes soulevés. -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. 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 | | **Relecture en diagonale** | 1 | 2 | 3 |