Skip to content

Commit 05fa68a

Browse files
committed
Tighten the open pull request limit guide wording
Review feedback: drop statements that are either common knowledge or likely to age, and keep the "help in other ways" list to contributions that do not need a pull request of their own. Generated-by: Claude Opus 5
1 parent 215d6b9 commit 05fa68a

1 file changed

Lines changed: 3 additions & 5 deletions

File tree

‎contributing-docs/32_open_pull_request_limit.rst‎

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -32,9 +32,9 @@ Reviewer attention is the scarcest resource in the project. When we had around 1
3232
Requests, we could promise - and mostly delivered - at least an initial review for each of them.
3333
With more than 1,000 open Pull Requests, that promise is impossible to keep, and hundreds of Pull
3434
Requests sit without any review at all. Opening more Pull Requests does not make any of them get
35-
merged faster - it only makes the queue longer for everyone, including you.
35+
merged faster - it only makes the queue longer for everyone.
3636

37-
Generating code has also become fast and cheap. A submitted Pull Request no longer shows on its own
37+
Generating code has also become fast. A submitted Pull Request no longer shows on its own
3838
that its author understands Airflow or is invested in the change, and deciding whether a Pull Request
3939
deserves a deeper review takes time that maintainers do not have for a queue of that size.
4040

@@ -55,8 +55,7 @@ How the limit works
5555
* The limit uses GitHub's
5656
`pull request limit for users without write access <https://docs.github.com/en/communities/moderating-comments-and-conversations/limiting-interactions-in-your-repository#limiting-concurrent-open-pull-requests-for-users-without-write-access>`__,
5757
enabled in the repository's ``.asf.yaml`` file.
58-
* It applies to every user **without write access** to the repository. Committers are not affected.
59-
Triage collaborators do not have write access, so the limit applies to them as well.
58+
* It applies to every user **without write access** to the repository.
6059
* Draft Pull Requests do not count towards the limit yet, because GitHub does not support counting
6160
them. Once GitHub adds that, we will start counting drafts as well - so do not rely on drafts to
6261
get around the limit. Keep unfinished work on a branch in your fork instead.
@@ -124,7 +123,6 @@ and they are what maintainers look at when deciding who to invite as a committer
124123
* Join the discussions on the ``[email protected]`` mailing list and in the dev calls.
125124
* Help other contributors on `Slack <https://s.apache.org/airflow-slack>`__, for example in the
126125
``#new-contributors`` channel.
127-
* Improve the documentation.
128126

129127
See `How to contribute <04_how_to_contribute.rst>`__ and
130128
`How to communicate <02_how_to_communicate.rst>`__ for more ways to get involved.

0 commit comments

Comments
 (0)