baalajimaestro/traefik
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages 1 Wiki Activity Actions

Update PR approval process

Browse source
This commit is contained in:
Emile Vauge 2024-07-10 11:46:03 +02:00 committed by GitHub
parent 927f0bc01a
commit f32884d9b8
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
1 changed files with 2 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
docs/content/contributing/submitting-pull-requests.md
View file

@ -56,7 +56,7 @@ Merging a PR requires the following steps to be completed before it is merged au
* Ensure your PR is not a draft. We do not review drafts, but do answer questions and confer with developers on them as needed. * Ensure your PR is not a draft. We do not review drafts, but do answer questions and confer with developers on them as needed.
* Pass the validation check. * Pass the validation check.
* Pass all tests. * Pass all tests.
* Receive 3 approving reviews from maintainers. * Receive 2 approving reviews from maintainers.
## Pull Request Review Cycle ## Pull Request Review Cycle
@ -112,7 +112,7 @@ In such a situation, solve the conflicts/CI/... and then remove the label `bot/n
To prevent the bot from automatically merging a PR, add the label `bot/no-merge`. To prevent the bot from automatically merging a PR, add the label `bot/no-merge`.
The label `bot/light-review` decreases the number of required LGTM from 3 to 1. The label `bot/light-review` decreases the number of required LGTM from 2 to 1.
This label can be used when: This label can be used when: