Intermédiaire · 6 min de lecture
Bien relire une pull request
La tester en local, commenter utilement, approuver ou demander des changements — et recevoir une review sans souffrir.
Récupérer la PR sur ta machine
Lire le diff ne suffit pas toujours : lance-la. Avec la CLI GitHub ou avec Git seul (le numéro de la PR est dans son URL).
$ gh pr checkout 123# or: git fetch origin pull/123/head:pr-123 && git switch pr-123
Ce qu'il faut regarder
Est-ce que ça fait ce que dit la description ? Cas limites, erreurs, sécurité (secrets, entrées utilisateur), tests, noms qui s'expliquent seuls. Le style : laisse le linter s'en occuper.
Commenter comme un humain
Pose des questions (« que se passe-t-il si… ? »), explique pourquoi, et dis ce qui bloque (à corriger) et ce qui est optionnel (nit:). La « Suggestion » de GitHub laisse l'auteur appliquer ta correction en un clic.
Voir ce qui a changé depuis ta dernière review
Après un push forcé, range-diff compare l'ancienne et la nouvelle version de la branche, commit par commit.
$ git range-diff origin/main old-head new-head
Recevoir une review
Les commentaires portent sur le code, pas sur toi. Réponds à chacun, pousse les corrections en nouveaux commits pendant la review, et redemande une review quand c'est fait.