Configure pull request limit for users without write access
Une pull request liée a déjà été fusionnée.
- #65250 par @mcollina — fusionnée
Évaluation
- Difficulté
- 5/5
- Temps estimé
- Plus d'une semaine
- Accessibilité débutants
- 25/100
- Type d'issue
- Fonctionnalité
- Clarté
- À clarifier
- Activité
- À l'abandon
- Stack technique
- github
- Domaine
- developer-experience
Piste de recherche
Commencez par consulter la documentation liée sur la limite de pull requests de GitHub et la discussion avec les contributeurs dans cette issue. C’est terminé lorsqu’une décision de la communauté concernant la limite a été prise et que le paramètre de dépôt sélectionné a été documenté ou appliqué.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
The open pull request count on nodejs/node has grown beyond 1000+. A portion of these are from individual contributors who have a high number of open PRs at the same time, which places demands on finite collaborator review capacity.
GitHub provides a setting to limit the number of open pull requests from users without write access. This helps balance the need for open contribution with the reality of limited human review resources.
Proposal
Should we set a pull request limit for users without write access?
This idea was discussed among collaborators, and 5 was suggested as a reasonable starting point.
We'd like to hear broader community input before making a decision.
Rationale
- Every open PR requires collaborator time for review, feedback, and follow-up. A high volume of open PRs from a single contributor can reduce the time available to review contributions from others.
- With the rise of AI-assisted tooling, it is now easier than ever to generate pull requests at a higher frequency, which can outpace the capacity of human reviewers.
- A limit encourages contributors to prioritize their most impactful changes and work with collaborators to get existing PRs merged or closed before opening new ones.
- It also serves as a backstop against automated spamming attacks that open multiple PRs.
- Contributors who are consistently engaged at a level that requires more open PRs would be candidates for collaborator status, which is not subject to the limit.
Notes
- The limit applies only to users without write access. Collaborators are not affected.
- We can start with 5 (or some other number based on consensus) and re-evaluate based on feedback.
- GitHub also provides a "bypass list" but we propose not using it to avoid the overhead of managing additions and removals.
- Langage dominant
- JavaScript
- Étoiles
- 202
- Forks
- 183
- Merge moyen
- 13 j 12 h
- PR mergées (30 j)
- 2
Guide de contribution
Ouvrir le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de nodejs/admin
-
Difficulté 1/5 Moins d'une heure Accessibilité débutants 90/100
-
tsc-agenda
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 30/100
-
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 35/100
-
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 35/100
-
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 30/100
Toutes les issues de nodejs/admin
Issues similaires
-
Difficulté 2/5 1-3 heures Accessibilité débutants 82/100
-
area-deployment area-integrations triage:bot-seen
Difficulté 2/5 Une demi-journée Accessibilité débutants 86/100
-
Issue-Bug
Difficulté 2/5 1-3 heures Accessibilité débutants 68/100
sugarlabs/musicblocks#8924 ·
-
Difficulté 1/5 Moins d'une heure Accessibilité débutants 90/100
ArduPilot/ardupilot_wiki#8088 ·
-
[BUG] createTool tools cannot be registered with Mastra when exactOptionalPropertyTypes is enabled Ouvertecustomer-eng status: needs triage
Difficulté 2/5 1-3 heures Accessibilité débutants 84/100