This is an automated email from the ASF dual-hosted git repository.
hubcio pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/iggy.git
The following commit(s) were added to refs/heads/master by this push:
new 72e46049d docs: welcome PR reviews from contributors (#4494)
72e46049d is described below
commit 72e46049d388408a8d4627e8a8a19501eadea454
Author: Justin Mclean <[email protected]>
AuthorDate: Mon Oct 12 12:54:06 2026 +0100
docs: welcome PR reviews from contributors (#4494)
---
CONTRIBUTING.md | 13 +++++++++++++
1 file changed, 13 insertions(+)
diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md
index c7d5e04e5..6679037c9 100644
--- a/CONTRIBUTING.md
+++ b/CONTRIBUTING.md
@@ -189,6 +189,17 @@ as described in `rfcs/README.md`. If no scope in
`.github/workflows/pr-title.yml` fits, add the RFC's feature name to that list
in the same pull request.
+## Reviewing PRs
+
+Anyone can review a PR. You don't need to be a committer, and reviews from
contributors
+are welcome. Read the change, run it locally if you can, and leave comments or
a review.
+Questions count too. If something isn't clear to you, it probably isn't clear
to the next
+reader either.
+
+A committer still has to approve and merge. If you're a returning contributor,
your
+"Request changes" review moves the PR to `S-waiting-on-author`, and you can
use `/author`
+to do the same.
+
## PR Triage Commands
Move a PR around the review queue by posting a slash command on its own
@@ -202,6 +213,8 @@ line in a regular PR comment (not an inline review reply):
| `/pin` | author or maintainer |
add `pinned`, exempting the PR from the stale bot |
| `/unpin` | author or maintainer |
remove `pinned` |
+A returning contributor is anyone who has had a commit merged into apache/iggy.
+
Some labels move on their own. Opening a non-draft PR, or marking a draft
ready, sets `S-waiting-on-review`. A "Request changes" review sets
`S-waiting-on-author`, and so does a `/skill` bot review with findings for the
