Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Commit bc97caa

Browse files
committed
docs: corrige la présentation des exemples de revue de code
1 parent 17ddeab commit bc97caa

1 file changed

Lines changed: 8 additions & 8 deletions

File tree

module-07-bonnes-pratiques-et-workflows/06-revue-de-code-avec-pr.md

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -468,8 +468,8 @@ Pour des détails non bloquants :
468468
N'oubliez pas de souligner ce qui est bien fait !
469469

470470
```markdown
471-
✨ Excellente gestion des erreurs ! C'est très clair.
472-
👍 Bonne idée d'extraire cette logique dans une fonction séparée.
471+
✨ Excellente gestion des erreurs ! C'est très clair.
472+
👍 Bonne idée d'extraire cette logique dans une fonction séparée.
473473
🎉 Les tests sont vraiment complets, bravo !
474474
```
475475

@@ -790,13 +790,13 @@ Si les changements sont importants, demandez au reviewer de revoir.
790790
**Pour `main`** :
791791

792792
```yaml
793-
✅ Require a pull request before merging
794-
✅ Require approvals: 2
795-
✅ Dismiss stale reviews when new commits are pushed
793+
✅ Require a pull request before merging
794+
✅ Require approvals: 2
795+
✅ Dismiss stale reviews when new commits are pushed
796796
✅ Require review from Code Owners
797797

798-
✅ Require status checks to pass before merging
799-
✅ Require branches to be up to date before merging
798+
✅ Require status checks to pass before merging
799+
✅ Require branches to be up to date before merging
800800
☑️ Status checks: tests, lint, build
801801

802802
✅ Require conversation resolution before merging
@@ -811,7 +811,7 @@ Si les changements sont importants, demandez au reviewer de revoir.
811811
**Pour `develop`** :
812812

813813
```yaml
814-
✅ Require a pull request before merging
814+
✅ Require a pull request before merging
815815
✅ Require approvals: 1
816816

817817
✅ Require status checks to pass before merging

0 commit comments

Comments
 (0)