Rédiger une description de PR qu'un relecteur peut utiliser
Objectif : permettre à un relecteur de décider en deux minutes ce qu'il doit regarder, et permettre à un futur lecteur de comprendre pourquoi la modification existe.
Collecte
- Trouvez la branche de base et exécutez
git log --oneline <base>..HEADetgit diff --stat <base>...HEAD, puis lisez le diff. - Cherchez la raison : un numéro de ticket ou d'issue lié dans les commits ou le nom de la branche, la demande de l'utilisateur, des commentaires ajoutés dans le diff. Si la raison ne se trouve dans aucun de ces éléments, demandez à l'utilisateur ; n'inventez pas de motivation.
- Vérifiez ce qui a été testé : les fichiers de test dans le diff, et les commandes que l'utilisateur a exécutées. Si les tests n'ont pas été lancés, dites-le clairement.
- Recherchez un fichier
.github/pull_request_template.mdouPULL_REQUEST_TEMPLATE. S'il en existe un, remplissez-le au lieu d'utiliser la structure ci-dessous.
Structure
<Titre : impératif, moins de 70 caractères, nomme le changement de comportement, pas les fichiers>
## Pourquoi
<Le problème ou la demande, en 1 à 3 phrases. Liez l'issue s'il y en a une.>
## Quoi de changé
- <Points au niveau du comportement regroupés par domaine, pas une liste de fichiers.>
## Comment relire
- Commencez par <fichier ou domaine> ; le reste en découle.
- Partie risquée : <la partie la plus susceptible d'être erronée, et pourquoi>.
- Pas changé volontairement : <tout ce qu'un relecteur pourrait attendre et ne trouvera pas>.
## Tests
- <Commandes exactes exécutées et résultats.>
- <Étapes manuelles et ce qui a été observé. "Non testé : ..." le cas échéant.>
## Déploiement/risque
<Migrations, configuration, feature flags, compatibilité, comment revenir en arrière. "Aucun" seulement si c'est vrai.>
Règles
- Décrivez ce que le diff fait, pas ce que vous souhaitez qu'il fasse. Relisez le diff après avoir rédigé et supprimez toute affirmation qu'il ne soutient pas.
- Ne listez pas chaque fichier. Listez les changements de comportement, la nouvelle configuration, les nouvelles dépendances et le code supprimé.
- Pas de remplissage ("Cette PR vise à..."). Pas d'émojis sauf si le dépôt en utilise.
- Captures d'écran ou exemples de sortie : dites où cela serait utile ; ne décrivez jamais une capture d'écran que vous n'avez pas vue.
- Affichez le titre et le corps dans un bloc de code afin qu'ils puissent être copiés. N'exécutez pas
gh pr createsauf si on vous le demande.