diff options
| author | Loek Le Blansch <loek@pipeframe.xyz> | 2024-12-19 20:19:38 +0100 |
|---|---|---|
| committer | Loek Le Blansch <loek@pipeframe.xyz> | 2024-12-19 20:19:38 +0100 |
| commit | 1a2e4005e2e164b81f229210836df794cc0ba9fa (patch) | |
| tree | 45a5874719f0e3f9be367a12464509e81eda85f1 /contributing.md | |
| parent | c737b01035cb62712b1dabcfef6f5014bdc882d1 (diff) | |
update contributing.md
Diffstat (limited to 'contributing.md')
| -rw-r--r-- | contributing.md | 14 |
1 files changed, 14 insertions, 0 deletions
diff --git a/contributing.md b/contributing.md index 5555892..d617a66 100644 --- a/contributing.md +++ b/contributing.md @@ -19,6 +19,20 @@ that you can click on to open them. an explanation as to why the code can not (reliably) be tested - Non-bugfix pull requests must be approved by at least 2 reviewers before being merged +- Pull requests should have the following labels (where appropriate) + |label|meaning| + |:-:|-| + |`fix me`|has feedback that should be resolved/discussed by its author| + |`review me`|needs additional reviewers (minimum of 2 per PR)| + |`do not review`|is actively being worked on or not ready for feedback| + |`high priority`|should be worked on before all the others| + - PRs start with the `review me` label + - Reviewers— + - Add the `fix me` label after adding comments + - Authors— + - Remove the `review me` label if the pull request has enough reviewers + - Add the `do not review` label while processing feedback / pushing + additional commits <!-- - TODO: tagging / versions |