Opened 3 weeks ago

Closed 2 weeks ago

#37426 closed Cleanup/optimization (fixed)

Restrict the allowed number of open PRs to django/django for new contributors

Reported by: Sarah Boyce Owned by: Sarah Boyce
Component: Core (Other) Version: dev
Severity: Normal Keywords:
Cc: Raffaella Triage Stage: Ready for checkin
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

Following discussions with many Django contributors, we have agreed that Django should use a GitHub Action to limit the number of open pull requests from new contributors.

When a pull request is opened on django/django, the action will check whether the contributor meets all three conditions:

  • Has fewer than five commits on Django’s main branch.
  • Has another open pull request created within the past six months.
  • Is not a member of a Django organisation GitHub team, such as the triage and review team, accessibility team, or steering council.

If all three conditions hold, the new pull request will be automatically closed with an explanatory message.

Thank you for contributing to Django!

Since you have fewer than five commits in Django and already have an open pull request created within the past six months, please finish or close that pull request before opening another. We are closing this pull request because you have reached this limit.

Please focus on your existing pull request and any outstanding review feedback. If you would prefer to work on this issue, you can close your existing pull request and unassign yourself from its associated ticket so someone else can take it on. You can then reopen this pull request.

The aim is to help new contributors learn Django’s contribution and review process by focusing on one contribution at a time, responding to feedback, and bringing their work to completion. This would also discourage contributors from claiming several tickets at once, leaving them unavailable to others.

The limits that I have suggested (5 commits, 6 months etc) can be later changed or changed during review.

Change History (7)

comment:1 by Sarah Boyce, 3 weeks ago

Owner: set to Sarah Boyce
Status: new → assigned

comment:2 by Natalia Bidart, 3 weeks ago

Triage Stage: Unreviewed → Accepted
Version: 6.1 → dev

Thank you! This matches our conversations.

comment:3 by Raffaella, 3 weeks ago

Cc: Raffaella added

comment:4 by Sarah Boyce, 2 weeks ago

Has patch: set

comment:5 by Jacob Walls, 2 weeks ago

Triage Stage: Accepted → Ready for checkin

Meets requirements and adds smart refactors. All my minor feedback is optional.

comment:6 by Sarah Boyce <42296566+sarahboyce@…>, 2 weeks ago

In 2a79923:

Refs #37426 -- Moved scripts GitHub API helpers to a shared module for reuse.

comment:7 by Sarah Boyce <42296566+sarahboyce@…>, 2 weeks ago

Resolution: → fixed
Status: assigned → closed

In 9b83f75:

Fixed #37426 -- Added a pull request limit for new contributors.

Note: See TracTickets for help on using tickets.
Back to Top