Skip to content

Close PRs opened from orgs or bot accounts#10281

Merged
lynxplay merged 2 commits into
PaperMC:masterfrom
lynxplay:enhancement/close-org-opened-prs
Feb 28, 2024
Merged

Close PRs opened from orgs or bot accounts#10281
lynxplay merged 2 commits into
PaperMC:masterfrom
lynxplay:enhancement/close-org-opened-prs

Conversation

@lynxplay
Copy link
Copy Markdown
Contributor

Expands the existing close_invalid_prs workflow to also close prs that were opened from repositories not owned by user accounts. This is required as such PRs cannot be edited by maintainers.

The implementation re-uses the existing job instead of creating a new job to limit the amount of checks to the bare minimum.

Expands the existing close_invalid_prs workflow to also close prs that
were opened from repositories not owned by user accounts.
This is required as such PRs cannot be edited by maintainers.

The implementation re-uses the existing job instead of creating a new
job to limit the amount of checks to the bare minimum.
@lynxplay lynxplay requested a review from a team as a code owner February 23, 2024 19:32
Copy link
Copy Markdown
Member

@electronicboy electronicboy left a comment

Choose a reason for hiding this comment

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

approving on semantic analysis, no means to test

Comment thread .github/workflows/close_invalid_prs.yml Outdated
@lynxplay lynxplay merged commit 850b736 into PaperMC:master Feb 28, 2024
@lynxplay lynxplay deleted the enhancement/close-org-opened-prs branch February 28, 2024 14:19
LeonTG pushed a commit to LeonTG/Paper that referenced this pull request May 17, 2026
Expands the existing close_invalid_prs workflow to also close prs that
were opened from repositories not owned by user accounts.
This is required as such PRs cannot be edited by maintainers.

The implementation re-uses the existing job instead of creating a new
job to limit the amount of checks to the bare minimum.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants