Skip to content

Restore the open pull request creation cap - #73758

Merged
shahar1 merged 1 commit into
apache:mainfrom
potiuk:restore-open-pr-creation-cap
Sep 26, 2026
Merged

shahar1 merged 1 commit into
apache:mainfrom
potiuk:restore-open-pr-creation-cap

Conversation

@potiuk

@potiuk potiuk commented Sep 26, 2026

Copy link
Copy Markdown
Member

Re-enables the creation_cap setting in .asf.yaml (max 5 open PRs for users without write access), reverting #73754. That PR removed the cap temporarily because enabling it failed while the asfyaml version supporting creation_cap was not deployed yet (INFRA-28454: https://issues.apache.org/jira/browse/INFRA-28454).


Was generative AI tooling used to co-author this PR?
  • Yes — Claude Code (Opus 5)

Generated-by: Claude Code (Opus 5) following the guidelines

🤖 Generated with Claude Code

The cap was temporarily removed in apache#73754 because enabling
`creation_cap` failed while the asfyaml version supporting it was not
deployed (INFRA-28454). This turns the open pull request limit back into
a hard limit enforced by GitHub instead of a soft limit contributors are
only asked to respect.

This reverts commit 7e5c580.

Generated-by: Claude Opus 5
@shahar1
shahar1 merged commit f863982 into apache:main Sep 26, 2026
65 checks passed
@potiuk
potiuk deleted the restore-open-pr-creation-cap branch September 26, 2026 18:07
dwsmith1983 pushed a commit to dwsmith1983/datafusion that referenced this pull request Oct 4, 2026
## Which issue does this PR close?

Follows [DataFusion discussion
apache#24413](apache#24413).

## Rationale for this change

See discussion for the rationale, most people are voting for 'set limit
to 3' from the discussion.

## What changes are included in this PR?

Enable `github.pull_requests.creation_cap` in `.asf.yaml` and set
`max_open_pull_requests` to `3`.

- ASF support:
[apache/infrastructure-asfyaml#111](apache/infrastructure-asfyaml#111).
- Airflow example followed by this change:
[apache/airflow#73758](apache/airflow#73758).

Whitelist support through `.asf.yaml` is being added in
[apache/infrastructure-asfyaml#135](apache/infrastructure-asfyaml#135),
which is still open. Before that I think it's possible to do some manual
close/reopen, if some contributor have > 3 concurrent PRs that is
mergeable.

## What is the testing strategy for this PR?

Should work since it's following the pattern in other ASF project above

I don't know if it's possible to test before merging

## Are there any user-facing changes?

NA
Omega359 pushed a commit to Omega359/arrow-datafusion that referenced this pull request Oct 11, 2026
## Which issue does this PR close?

Follows [DataFusion discussion
apache#24413](apache#24413).

## Rationale for this change

See discussion for the rationale, most people are voting for 'set limit
to 3' from the discussion.

## What changes are included in this PR?

Enable `github.pull_requests.creation_cap` in `.asf.yaml` and set
`max_open_pull_requests` to `3`.

- ASF support:
[apache/infrastructure-asfyaml#111](apache/infrastructure-asfyaml#111).
- Airflow example followed by this change:
[apache/airflow#73758](apache/airflow#73758).

Whitelist support through `.asf.yaml` is being added in
[apache/infrastructure-asfyaml#135](apache/infrastructure-asfyaml#135),
which is still open. Before that I think it's possible to do some manual
close/reopen, if some contributor have > 3 concurrent PRs that is
mergeable.

## What is the testing strategy for this PR?

Should work since it's following the pattern in other ASF project above

I don't know if it's possible to test before merging

## Are there any user-facing changes?

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants