LadybirdBrowser/ladybird
1
1
Fork 0
mirror of https://github.com/LadybirdBrowser/ladybird.git synced 2024-08-17 01:10:28 +03:00
Code Issues Packages Projects Releases Wiki Activity

Meta: Update CONTRIBUTING.md

Browse Source 
- Rename "reviewers" to "maintainers" to disambiguate, since everyone
  is encouraged to participate in code review.

- Clarify that maintainership is by invitation only and unrelated to
  participation metrics.
This commit is contained in:
Andreas Kling 2021-07-09 00:44:18 +02:00
parent c94b0e80e7
commit 36c8b1c58c
Notes: sideshowbarker 2024-07-18 10:02:20 +09:00 
Author: https://github.com/awesomekling
Commit: https://github.com/SerenityOS/serenity/commit/36c8b1c58ca
1 changed files with 8 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

14
CONTRIBUTING.md
View File

@ -65,17 +65,19 @@ Nobody is perfect, and sometimes we mess things up. That said, here are some goo
## Pull Request Q&A
### I've submitted a PR and it passes CI. When can I expect to get first reviewer feedback?
### I've submitted a PR and it passes CI. When can I expect to get some review feedback?
While unadvertised PR's may get randomly merged by curious reviewers, you will have a much smoother time if you engage with the community on Discord.
While unadvertised PR's may get randomly merged by curious maintainers, you will have a much smoother time if you engage with the community on Discord.
### If my PR isn't getting attention, how long should I wait before pinging one of the project reviewers?
### If my PR isn't getting attention, how long should I wait before pinging one of the project maintainers?
Ping them right away if it's something urgent! If it's less urgent, advertise your PR on Discord and ask if someone could review it.
Ping them right away if it's something urgent! If it's less urgent, advertise your PR on Discord (`#code-review`) and ask if someone could review it.
### Who are the project reviewers?
### Who are the project maintainers?
The project reviewers at this time are [@awesomekling](https://github.com/awesomekling), [@linusg](https://github.com/linusg), [@alimpfard](https://github.com/alimpfard), and [@gunnarbeutner](https://github.com/gunnarbeutner).
The project maintainers at this time are [@awesomekling](https://github.com/awesomekling), [@linusg](https://github.com/linusg), [@alimpfard](https://github.com/alimpfard), and [@gunnarbeutner](https://github.com/gunnarbeutner).
Maintainership is by invitation only and does not correlate with any particular metric.
### Is there a policy for branches/PRs that haven't been touched in X days? Should they be closed?