Skip to content

ci: Fix PR limiter association exemptions#390

Merged
peterj merged 1 commit intoagentregistry-dev:mainfrom
timflannagan:ci/pr-limit-author-assoc-csv
Mar 20, 2026
Merged

ci: Fix PR limiter association exemptions#390
peterj merged 1 commit intoagentregistry-dev:mainfrom
timflannagan:ci/pr-limit-author-assoc-csv

Conversation

@timflannagan
Copy link
Copy Markdown
Collaborator

Description

This updates the PR limiter workflow to pass exempt author associations in the format the pinned action parses.

Previously, we passed a newline-separated YAML block. The action splits this input on commas, so maintainers were still limited.

Now, we pass a comma-separated list.
MEMBER, OWNER, and COLLABORATOR are correctly exempted.

Fixes #387.

Change Type

/kind cleanup

Changelog

NONE

Additional Notes

This updates the PR limiter workflow to pass exempt author associations
in the format the pinned action parses.

Previously, we passed a newline-separated YAML block.
The action splits this input on commas, so maintainers were still limited.

Now, we pass a comma-separated list.
MEMBER, OWNER, and COLLABORATOR are correctly exempted.
Copilot AI review requested due to automatic review settings March 19, 2026 20:58
@github-actions
Copy link
Copy Markdown

You already have 3 pull requests open. Please consider working on getting the existing ones merged before opening new ones. Thanks!

# this workflow limits the number of open PRs created by a user
name: PR Limiter
on:
pull_request_target:
Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

FYI - this event trigger type forces us to merge this before we can validate the changes work

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates the PR limiter workflow configuration so exempt GitHub author associations are passed in the comma-separated format expected by the pinned Homebrew/actions/limit-pull-requests action, preventing maintainers/members from being incorrectly limited.

Changes:

  • Change except-author-associations from a newline-separated YAML block to a comma-separated string (MEMBER,OWNER,COLLABORATOR).

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@peterj peterj added this pull request to the merge queue Mar 20, 2026
Merged via the queue into agentregistry-dev:main with commit dadfebe Mar 20, 2026
12 checks passed
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.

The pr-limit workflow doesn't set except-author-associations correctly

3 participants