Skip to content

Remove branch restrictions from PR workflow triggers#309

Merged
knoepfel merged 2 commits intomainfrom
copilot/remove-main-target-restriction
Feb 10, 2026
Merged

Remove branch restrictions from PR workflow triggers#309
knoepfel merged 2 commits intomainfrom
copilot/remove-main-target-restriction

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Feb 10, 2026

PR checks were restricted to only run when targeting main or develop branches. This prevented CI validation on PRs targeting feature branches, release branches, or any other branch.

Changes

Removed branches: [ main, develop ] from pull_request: triggers in 10 workflow files:

  • actionlint-check.yaml
  • clang-format-check.yaml
  • clang-tidy-check.yaml
  • cmake-build.yaml
  • cmake-format-check.yaml
  • codeql-analysis.yaml
  • coverage.yaml
  • jsonnet-format-check.yaml
  • markdown-check.yaml
  • python-check.yaml

Before:

on:
  pull_request:
    branches: [ main, develop ]

After:

on:
  pull_request:

Branch restrictions on push: triggers remain unchanged.


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Removed `branches: [ main, develop ]` from `pull_request:` triggers in 10 workflow files to allow checks to run on PRs targeting any branch, not just main and develop.

Co-authored-by: greenc-FNAL <2372949+greenc-FNAL@users.noreply.github.com>
Copilot AI changed the title [WIP] Remove restriction for workflows triggering on PRs Remove branch restrictions from PR workflow triggers Feb 10, 2026
Copilot AI requested a review from greenc-FNAL February 10, 2026 16:44
@greenc-FNAL greenc-FNAL marked this pull request as ready for review February 10, 2026 16:46
Copilot AI review requested due to automatic review settings February 10, 2026 16:46
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

This PR updates GitHub Actions workflow triggers so CI checks run for pull requests targeting any branch (not just main/develop), improving validation coverage for feature/release branch workflows.

Changes:

  • Removed branches: [ main, develop ] filters from the pull_request: trigger in 10 workflow files.
  • Kept push: branch restrictions unchanged.

Reviewed changes

Copilot reviewed 10 out of 10 changed files in this pull request and generated no comments.

Show a summary per file
File Description
.github/workflows/actionlint-check.yaml Allows actionlint checks to run on PRs targeting any branch.
.github/workflows/clang-format-check.yaml Allows clang-format checks to run on PRs targeting any branch.
.github/workflows/clang-tidy-check.yaml Allows clang-tidy checks to run on PRs targeting any branch.
.github/workflows/cmake-build.yaml Allows build/test workflow to run on PRs targeting any branch.
.github/workflows/cmake-format-check.yaml Allows cmake-format checks to run on PRs targeting any branch.
.github/workflows/codeql-analysis.yaml Allows CodeQL analysis to run on PRs targeting any branch.
.github/workflows/coverage.yaml Allows coverage workflow to run on PRs targeting any branch.
.github/workflows/jsonnet-format-check.yaml Allows jsonnet format checks to run on PRs targeting any branch.
.github/workflows/markdown-check.yaml Allows markdown checks to run on PRs targeting any branch.
.github/workflows/python-check.yaml Allows python checks to run on PRs targeting any branch.

@greenc-FNAL greenc-FNAL requested a review from knoepfel February 10, 2026 16:52
@knoepfel knoepfel merged commit fe24169 into main Feb 10, 2026
51 checks passed
@knoepfel knoepfel deleted the copilot/remove-main-target-restriction branch February 10, 2026 18:58
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.

4 participants