Vocabulaire code review et pull request en anglais

Demande de fusion

“I opened a PR for the new authentication flow.”

Contexte : Sur GitHub/Bitbucket. Sur GitLab, on dit 'merge request' (MR).

Demande de fusion (GitLab)

“The MR has been open for 3 days — can someone review?”

Contexte : Équivalent GitLab de la pull request GitHub.

Branche

“Create a feature branch from main before starting work.”

Contexte : Copie parallèle du code pour travailler sans impacter la branche principale.

Différences (entre deux versions)

“The diff is pretty clean — only 50 lines changed.”

Contexte : Ce qui a changé entre votre branche et la branche cible.

Commit (enregistrement de modifications)

“Please squash your commits before merging.”

Contexte : Un snapshot de vos modifications avec un message descriptif.

Message de commit

“Write meaningful commit messages — not just 'fix stuff'.”

Contexte : Description de ce que fait le commit. Convention : impératif, concis.

Description (de la PR)

“Add a clear description explaining what the PR does and why.”

Contexte : Le texte qui accompagne la PR — contexte, motivation, screenshots.

PR brouillon

“I opened a draft PR to get early feedback on the approach.”

Contexte : PR pas encore prête pour review — signale un travail en cours.

Revue de code

“Can someone pick up my code review? It's been open since yesterday.”

Contexte : Le processus d'examen du code par un pair avant fusion.

Relecteur / réviseur

“I assigned two reviewers to this PR.”

Contexte : La personne qui examine et commente votre code.

Approuver

“I approved the PR — ship it!”

Contexte : Donner son feu vert. Souvent il faut 1-2 approbations pour merger.

Demander des modifications

“I requested changes — the error handling needs to be improved.”

Contexte : Le reviewer bloque la PR jusqu'à ce que les corrections soient faites.

Commentaire (sans statut)

“I left a comment but didn't block the PR.”

Contexte : Feedback sans bloquer ni approuver — observation ou question.

Seconde revue

“I pushed the fixes — can you re-review?”

Contexte : Demander au reviewer de relire après vos corrections.

Validation finale

“The tech lead needs to sign off before we merge to production.”

Contexte : Approbation formelle, souvent du lead ou du senior.

Assigné (responsable de la PR)

“The assignee is responsible for addressing all review comments.”

Contexte : La personne responsable de la PR — généralement l'auteur.

Détail mineur / pinaillage

“Nit: this variable name could be more descriptive.”

Contexte : Commentaire non-bloquant. Le reviewer signale un point mineur.

Bloquant

“This is a blocker — we can't merge with a SQL injection vulnerability.”

Contexte : Problème critique qui doit être corrigé avant le merge.

Suggestion

“Suggestion: consider using a Map instead of an object here.”

Contexte : Proposition d'amélioration, pas obligatoire.

Pinaillage

“This is just a nitpick, feel free to ignore.”

Contexte : Synonyme de nit. Commentaire sur un point vraiment mineur.

Hors périmètre

“Good point, but that's out of scope for this PR.”

Contexte : Le commentaire est valide mais concerne un autre sujet.

À traiter plus tard

“Let's create a follow-up issue for this refactoring.”

Contexte : Reporter un changement à une PR ou un ticket futur.

Commentaire sur une ligne

“I left an inline comment on line 42 about the edge case.”

Contexte : Commentaire attaché à une ligne spécifique du diff.

Discussion (thread de commentaires)

“There are 3 unresolved conversations on this PR.”

Contexte : Fil de discussion sur un point précis du code.

Résoudre (une conversation)

“I resolved the conversation after pushing the fix.”

Contexte : Marquer un thread comme traité — le feedback a été adressé.

Fusionner

“I'll merge the branch once CI passes.”

Contexte : Intégrer les changements de votre branche dans la branche cible.

Écraser et fusionner

“Please squash and merge to keep the history clean.”

Contexte : Combiner tous les commits d'une PR en un seul avant le merge.

Rebaser

“Rebase your branch on main to resolve the conflicts.”

Contexte : Rejouer vos commits sur la dernière version de la branche cible.

Conflit de fusion

“There's a merge conflict in the config file — can you resolve it?”

Contexte : Deux branches ont modifié le même fichier de manière incompatible.

Intégration continue

“CI failed on the linting step — fix the formatting.”

Contexte : Pipeline automatique qui teste le code à chaque push.

Pipeline (CI/CD)

“The pipeline takes about 8 minutes to run.”

Contexte : Séquence d'étapes automatisées : build, test, lint, deploy.

Build vert (tous les tests passent)

“We need a green build before merging.”

Contexte : Tous les checks CI sont passés — prêt à merger.

Sélection de commit (picorer)

“Cherry-pick this fix into the release branch.”

Contexte : Appliquer un commit spécifique sur une autre branche.

Annuler (un commit/merge)

“We need to revert this PR — it broke the login flow.”

Contexte : Créer un commit inverse pour annuler des changements.

PR périmée / obsolète

“Close stale PRs that have been open for more than 30 days.”

Contexte : PR ouverte depuis trop longtemps sans activité.

Looks Good To Me (ça me va)

“LGTM — approved!”

Contexte : L'abréviation la plus courante en code review. = Approuvé.

Work In Progress (en cours)

“WIP: don't review yet, just pushing to save my work.”

Contexte : Signale que la PR n'est pas prête pour review.

Please Take A Look (jette un oeil)

“PTAL — ready for review now.”

Contexte : Demande de review, souvent sur Slack ou dans un commentaire.

Too Long; Didn't Read (résumé)

“TL;DR: refactored the auth module to use JWT instead of sessions.”

Contexte : Résumé rapide pour les gens pressés.

Request For Comments (demande d'avis)

“RFC: should we migrate to the new API before or after the release?”

Contexte : Demande d'avis ou de discussion sur une décision technique.

Acknowledged (bien reçu)

“ACK — I'll fix this in the next commit.”

Contexte : Accusé de réception. Le feedback a été lu et sera traité.