docs: fixe le flux de branches et rend la relecture obligatoire #49

Merged

florian merged 1 commit from lenaic/0-flux-de-branches into develop

2026-09-01 11:37:36 +00:00

lenaic commented

2026-09-01 11:32:08 +00:00

Owner

Ce que ça change

Deux points qui étaient flous et que chacun tranchait dans son coin.

Le flux de branches. Une branche de travail part de develop et y retourne.
main ne reçoit que develop, en fin de projet ou avant une démonstration.
Jusqu'ici certaines demandes visaient main, d'autres develop, alors que
develop est la branche par défaut.

La relecture. CONVENTIONS.md disait déjà que la relecture par un pair est
notée au critère 6, mais la forge acceptait une fusion sans aucune approbation.
Le réglage est corrigé : une approbation obligatoire sur main et sur develop,
un avis défavorable bloque, une approbation tombe si la branche change après.

À savoir

Les demandes #22 et #17 ont été fusionnées juste avant ce changement, sans
relecture, faute de relecteur en 24 heures. C'est une exception, elle est
annoncée au point du soir. Cette demande-ci est la première à passer sous la
nouvelle règle, ce qui la teste au passage.

Le relecteur n'est pas le Tech Lead par défaut, sinon il devient un goulot
d'étranglement. Prenez celui qui connaît la zone, ou le premier disponible.

## Ce que ça change Deux points qui étaient flous et que chacun tranchait dans son coin. **Le flux de branches.** Une branche de travail part de `develop` et y retourne. `main` ne reçoit que `develop`, en fin de projet ou avant une démonstration. Jusqu'ici certaines demandes visaient `main`, d'autres `develop`, alors que `develop` est la branche par défaut. **La relecture.** `CONVENTIONS.md` disait déjà que la relecture par un pair est notée au critère 6, mais la forge acceptait une fusion sans aucune approbation. Le réglage est corrigé : une approbation obligatoire sur `main` et sur `develop`, un avis défavorable bloque, une approbation tombe si la branche change après. ## À savoir Les demandes #22 et #17 ont été fusionnées juste avant ce changement, sans relecture, faute de relecteur en 24 heures. C'est une exception, elle est annoncée au point du soir. Cette demande-ci est la première à passer sous la nouvelle règle, ce qui la teste au passage. Le relecteur n'est pas le Tech Lead par défaut, sinon il devient un goulot d'étranglement. Prenez celui qui connaît la zone, ou le premier disponible.

lenaic added the

Kind/Documentation

Priority

High

labels

2026-09-01 11:32:08 +00:00

lenaic added 1 commit

2026-09-01 11:32:09 +00:00

docs: fixe le flux de branches et la relecture obligatoire

Intégration / Qualité du code Python (pull_request) Successful in 3s

Details

Intégration / Tests unitaires (pull_request) Successful in 3s

Details

Intégration / Aucun secret commité (pull_request) Successful in 3s

Details

fa6ab4285d

Les branches de travail partent de develop et y retournent, main ne reçoit
que develop. Une approbation devient obligatoire pour fusionner, sur les
deux branches, ce que CONVENTIONS.md affirmait déjà sans que la forge le
fasse respecter.