Skip to content

ci: Limit open, non-draft PRs to 3 for users without write access #3465

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
CTTY merged 3 commits into from
Oct 15, 2026
Merged

ci: Limit open, non-draft PRs to 3 for users without write access #3465

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
79670cb
enforce open pull request limit and add to CONTRIBUTING.md
mbutrovich Oct 15, 2026
7dde20a
trim comment length
mbutrovich Oct 15, 2026
0ee9983
Update CONTRIBUTING.md
mbutrovich Oct 15, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 4 additions & 0 deletions .asf.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -83,6 +83,10 @@ github:
allow_auto_merge: true
# auto-delete head branches after being merged
del_branch_on_merge: true
# Limit open, non-draft PRs per user without write access. See CONTRIBUTING.md.

@comphead comphead Oct 15, 2026

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

oh do they finally addressed that? I filed an initial issue for ASF INFRA to support this. need to check if it actually works

creation_cap:
enabled: true
max_open_pull_requests: 3
features:
wiki: false
issues: true
Expand Down
4 changes: 4 additions & 0 deletions CONTRIBUTING.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -38,6 +38,10 @@ First, thank you for contributing to Iceberg Rust! The goal of this document is

Once your changes are ready you must submit your branch as a [pull request](https://github.com/apache/iceberg-rust/pulls).

#### Open Pull Request Limit

With coding agents, the number of open PRs now far exceeds our review capacity. To help reviewers focus on fewer PRs, users without write access are limited to **3 open, non-draft PRs** at a time. If you've reached the limit, wait for existing PRs to be merged, close some of them, or convert some to drafts until you have fewer than three open PRs before opening another.

#### Title

The pull request title must follow the format outlined in the [conventional commits spec](https://www.conventionalcommits.org). [Conventional commits](https://www.conventionalcommits.org) is a standardized format for commit messages. iceberg-rust only requires this format for commits on the `main` branch. And because iceberg-rust squashes commits before merging branches, this means that only the pull request title must conform to this format.
Expand Down
Loading