Skip to content

Restrict block-remove-before-merge check to PRs targeting main#1898

Open
edburns wants to merge 1 commit into
mainfrom
edburns/make-block-remove-before-merge-only-act-on-base-main
Open

Restrict block-remove-before-merge check to PRs targeting main#1898
edburns wants to merge 1 commit into
mainfrom
edburns/make-block-remove-before-merge-only-act-on-base-main

Conversation

@edburns

@edburns edburns commented Jul 2, 2026

Copy link
Copy Markdown
Collaborator

Only block merging when the PR base branch is main. PRs targeting other branches (e.g. feature or release branches) are now a no-op for this check.

Only block merging when the PR base branch is main. PRs targeting
other branches (e.g. feature or release branches) are now a no-op
for this check.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@edburns edburns requested a review from a team as a code owner July 2, 2026 23:29
Copilot AI review requested due to automatic review settings July 2, 2026 23:29

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Restricts the “Block remove-before-merge paths” GitHub Actions workflow so it only enforces the check for pull requests targeting the main branch, making the check a no-op for PRs targeting other branches.

Changes:

  • Updated the workflow job-level condition to run only when the event is pull_request and the PR base branch is main.
Show a summary per file
File Description
.github/workflows/block-remove-before-merge.yml Narrows the enforcement condition so the job only runs for PRs whose base branch is main.

Review details

  • Files reviewed: 1/1 changed files
  • Comments generated: 0
  • Review effort level: Low

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.

2 participants