diff --git a/content/docs/guides/contribute/review-process.en.md b/content/docs/guides/contribute/review-process.en.md index a1f5dbf1e..b5cea3a67 100644 --- a/content/docs/guides/contribute/review-process.en.md +++ b/content/docs/guides/contribute/review-process.en.md @@ -1,6 +1,6 @@ Here's a suggested workflow. -**It is necessary to communicate via instant messaging (Matrix, Slack, etc.) in order to guarantee the smooth flow of PR validation.** +**It can be useful to communicate via instant messaging (Matrix, Slack, etc.) in order to guarantee the smooth flow of PR validation.** ```mermaid sequenceDiagram @@ -11,10 +11,10 @@ sequenceDiagram R->>A: Answers yes or no loop Loop between author and reviewer - R-->>+A: Comments, asks for changes + R-->>A: Comments, asks for changes A-->>R: Answers to comments or requested changes - A-->>-R: Makes necessary changes in dedicated "fixups" - R-->>A: Reviews changes + A-->>R: Makes necessary changes in dedicated "fixups" + R-->>A: Reviews, tests changes, and comments again R-->>A: Resolves requested changes/conversations if ok end diff --git a/content/docs/guides/contribute/review-process.fr.md b/content/docs/guides/contribute/review-process.fr.md index 7d23567e9..99dffaf57 100644 --- a/content/docs/guides/contribute/review-process.fr.md +++ b/content/docs/guides/contribute/review-process.fr.md @@ -1,6 +1,6 @@ Voici une proposition de flux de travail. -**Il est nécessaire de communiquer par messagerie instantanée (Matrix, Slack, etc.) afin de garantir le fonctionnement du flux de la validation d'une PR.** +**Il peut être utile de communiquer par messagerie instantanée (Matrix, Slack, etc.) afin de garantir le fonctionnement du flux de la validation d'une PR.** ```mermaid sequenceDiagram @@ -12,14 +12,14 @@ sequenceDiagram loop Boucle entre auteur et reviewer R-->>A: Commente, demande des changements - A-->>R: Répond à chaque commentaire/requested changes + A-->>R: Répond à chaque commentaire/demande de changement A-->>R: Corrige le code si nécessaire dans des « fixups » dédiés - R-->>A: Vérifie et commente à nouveau le code - R-->>A: Résoud les conversations/requested changes le cas échéant + R-->>A: Vérifie, teste, et commente à nouveau le code + R-->>A: Résout les conversations/demandes de changement le cas échéant end A->>R: Rebase si nécessaire R->>A: Vérifie l'historique des commits R->>A: Approuve ou ferme la PR Note left of R: Et fusionne si mainteneur -``` \ No newline at end of file +```