Skip to content

Add gallagher branch to CodeQL workflow triggers - #2

Open
StephHope wants to merge 1 commit into
gallagherfrom
shope/codeql-on-gallagher-branch
Open

Add gallagher branch to CodeQL workflow triggers#2
StephHope wants to merge 1 commit into
gallagherfrom
shope/codeql-on-gallagher-branch

Conversation

@StephHope

Copy link
Copy Markdown

Add gallagher to CodeQL's trigger branches

advanced-codeql.yml currently only scans main-enterprise (push and PR). Since gallagher is now this repo's default and control branch, none of our own changes ever get scanned — main-enterprise only ever sees genuine upstream commits, never anything we add on top.

This adds gallagher alongside main-enterprise (not replacing it), so:

  • main-enterprise keeps scanning incoming upstream content as it lands — a useful automated check on top of the tracking branch's review, which is provenance-only ("is this really from upstream?"), not a content/security review.
  • gallagher now scans everything Gallagher-authored: in-tree fixes (e.g. teams.js), workflow/config changes like this one, and future upstream integration merges — none of which main-enterprise's trigger would ever catch.

No functional change to what CodeQL analyzes, just where it runs. Also needed as a prerequisite for gallagher's CodeQL checks to be selectable as required status checks later, since GitHub only offers checks it's actually seen run.

@StephHope
StephHope requested a review from a team July 30, 2026 05:00
@StephHope
StephHope changed the base branch from main-enterprise to gallagher July 30, 2026 05:03
@StephHope StephHope closed this Jul 30, 2026
@StephHope StephHope reopened this Jul 30, 2026
@StephHope StephHope closed this Jul 30, 2026
@StephHope StephHope reopened this Jul 30, 2026
@StephHope
StephHope marked this pull request as draft July 30, 2026 21:50
@StephHope
StephHope marked this pull request as ready for review July 30, 2026 21:50
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.

1 participant