Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
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
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@ Continuously inspect automatic E2E results for `main`. Coordinate ownership and
2. Keep release operations out of scope. Never change, retag, publish, or otherwise touch a release, tag, or release artifact during this workflow. Route release work to the existing release workflow.
3. Confirm maintainer authorization. Merge only when the request grants it. Otherwise, leave the PR `approval-ready` and continue the loop.
4. Check Git and GitHub access. Follow [Git and GitHub Access Hard Stop](../_shared/git-github-hard-stop.md) on access failure.
5. Fetch trusted `origin/main`. Read its PR-limit policy with `git show origin/main:.github/workflows/pr-limit.yaml`. For a non-exempt author, do not create a claim that would exceed the 10-open-PR limit.
5. Fetch trusted `origin/main`. Read its PR-limit policy with `git show origin/main:.github/workflows/pr-limit.yaml`. For a non-exempt author, do not create a claim that would exceed that limit.

Do not declare success or end the loop because the queue is empty or the newest run passes. Wait for the next automatic `main` result.

Expand Down
8 changes: 5 additions & 3 deletions .github/workflows/pr-limit.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -27,6 +27,8 @@ jobs:
PR_NUMBER: ${{ github.event.pull_request.number }}
REPO: ${{ github.repository }}
run: |
LIMIT=5

# Core maintainers are exempt from the PR limit.
EXEMPT="ericksoa kjw3 jacobtomlinson cv jyaunches"
for user in $EXEMPT; do
Expand All @@ -40,10 +42,10 @@ jobs:

echo "Author $AUTHOR has $OPEN_COUNT open PR(s)"

if [ "$OPEN_COUNT" -gt 10 ]; then
if [ "$OPEN_COUNT" -gt "$LIMIT" ]; then
gh pr comment "$PR_NUMBER" --repo "$REPO" --body \
"This repository limits contributors to 10 open pull requests. Please close or merge existing PRs before opening new ones."
"This repository limits contributors to $LIMIT open pull requests. Please close or merge existing PRs before opening new ones."
gh pr close "$PR_NUMBER" --repo "$REPO"
echo "::error::PR closed — author $AUTHOR exceeds the 10 open PR limit"
echo "::error::PR closed — author $AUTHOR exceeds the $LIMIT open PR limit"
exit 1
fi
2 changes: 1 addition & 1 deletion AGENTS.md
Original file line number Diff line number Diff line change
Expand Up @@ -344,4 +344,4 @@ If the command trace contains no reviewer-request write, report the event as an
- Direct PRs follow `.github/PULL_REQUEST_TEMPLATE.md`; the managed documentation workflow uses its generated body
- PRs that change `scripts/prepare-dgx-station-host.sh` must include reviewable DGX Station test evidence identifying the tested commit, Station profile or scenario, result, and a supporting link. Any maintainer may review the evidence; without acceptable evidence, the PR is not ready to approve or merge. Treat the evidence as human-reviewed, not authenticated hardware provenance. Exceptional bypasses use existing repository governance and must document the reason on the PR.
- No secrets, API keys, or credentials committed
- Apply the 10-open-PR limit from `.github/workflows/pr-limit.yaml` only to accounts that the workflow does not exempt
- Apply the open PR limit from `.github/workflows/pr-limit.yaml` only to accounts that the workflow does not exempt
2 changes: 1 addition & 1 deletion CONTRIBUTING.md
Original file line number Diff line number Diff line change
Expand Up @@ -511,7 +511,7 @@ For Markdown docs routing, user-skill guidance, and release-prep documentation w

## Pull Requests

We welcome contributions. Every PR requires maintainer review before merge. Contributors may have up to 10 open PRs at one time.
We welcome contributions. Every PR requires maintainer review before merge. Contributors may have up to 5 open PRs at one time.
Core maintainers listed in `.github/workflows/pr-limit.yaml` are exempt from this limit.
Maintainers review pull requests according to project priority, security impact, release readiness, and reviewer availability.
PRs that solve issues with Priority set to Urgent or High are more likely to receive earlier review when maintainers have capacity.
Expand Down
Loading