You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
Commit 05fa68a
Browse filesBrowse the repository at this point in the historyBrowse files
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
Copy file name to clipboardExpand all lines: contributing-docs/32_open_pull_request_limit.rst
+3-5Lines changed: 3 additions & 5 deletions
Display the source diff
Display the rich diff
Original file line number
Diff line number
Diff line change
@@ -32,9 +32,9 @@ Reviewer attention is the scarcest resource in the project. When we had around 1
32
32
Requests, we could promise - and mostly delivered - at least an initial review for each of them.
33
33
With more than 1,000 open Pull Requests, that promise is impossible to keep, and hundreds of Pull
34
34
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.
36
36
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
38
38
that its author understands Airflow or is invested in the change, and deciding whether a Pull Request
39
39
deserves a deeper review takes time that maintainers do not have for a queue of that size.
40
40
@@ -55,8 +55,7 @@ How the limit works
55
55
* The limit uses GitHub's
56
56
`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>`__,
57
57
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.
60
59
* Draft Pull Requests do not count towards the limit yet, because GitHub does not support counting
61
60
them. Once GitHub adds that, we will start counting drafts as well - so do not rely on drafts to
62
61
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
124
123
* Join the discussions on the ``[email protected]`` mailing list and in the dev calls.
125
124
* Help other contributors on `Slack <https://s.apache.org/airflow-slack>`__, for example in the
126
125
``#new-contributors`` channel.
127
-
* Improve the documentation.
128
126
129
127
See `How to contribute <04_how_to_contribute.rst>`__ and
130
128
`How to communicate <02_how_to_communicate.rst>`__ for more ways to get involved.
0 commit comments