Skip to content

doc: document open pull request limit - #65250

Open
mcollina wants to merge 1 commit into
nodejs:mainfrom
mcollina:doc/limit-open-pull-requests
Open

doc: document open pull request limit#65250
mcollina wants to merge 1 commit into
nodejs:mainfrom
mcollina:doc/limit-open-pull-requests

Conversation

@mcollina

Copy link
Copy Markdown
Member

I previously set the maximum number of open pull requests for non-Collaborators to 30 to prevent excessive AI contributions and maintainer burnout. I want to reduce that limit to 10, and this change documents the new limit in the contribution guides.

Signed-off-by: Matteo Collina <hello@matteocollina.com>
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

Review requested:

  • @nodejs/tsc

@nodejs-github-bot nodejs-github-bot added the doc Issues and PRs related to the documentations. label Aug 12, 2026

@legendecas legendecas left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can the number be put in CONTRIBUTING.md only, so that we don't need updating 3 places if the number changes?

@avivkeller avivkeller left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What's the process for users currently with more than 10 PRs, should their overflow be closed, or grandfathered in?

@Renegade334

Copy link
Copy Markdown
Member

What's the process for users currently with more than 10 PRs, should their overflow be closed, or grandfathered in?

GH will not close them, the user is just prevented from opening more until the total drops to <10.

@Renegade334 Renegade334 added the meta Issues and PRs related to the general management of the project. label Aug 12, 2026

@evenstensberg evenstensberg left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@aduh95

aduh95 commented Aug 13, 2026

Copy link
Copy Markdown
Contributor

What's the process for users currently with more than 10 PRs, should their overflow be closed, or grandfathered in?

GH will not close them, the user is just prevented from opening more until the total drops to <10.

IIUC GitHub would let them open more than 30, but won't let them mark as Ready for Reviews more than 30 at a time:

@aduh95

aduh95 commented Aug 13, 2026

Copy link
Copy Markdown
Contributor

I suggest we revert the 30-PR limit and close this PR in favor of nodejs/admin#1087. The absolute number of non-draft PRs is not saying much about the quality of a contribution, the rate the PRs are open does. The fact that GH UI indicates the limit so prominently can give users the impression that having up to 30 open PRs is OK, so IMO it's only making things worse. I'd rather have them open 50+ PRs at once so we can block them.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

doc Issues and PRs related to the documentations. meta Issues and PRs related to the general management of the project.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Configure pull request limit for users without write access