Skip to content

Run CodeQL on chart release-branch pull requests and pushes - #70727

Merged
Lee-W merged 1 commit into
apache:chart/v1-2x-testfrom
astronomer:codeql-trigger-chart-v1-2x-test
Jul 31, 2026
Merged

Run CodeQL on chart release-branch pull requests and pushes#70727
Lee-W merged 1 commit into
apache:chart/v1-2x-testfrom
astronomer:codeql-trigger-chart-v1-2x-test

Conversation

@Lee-W

@Lee-W Lee-W commented Jul 30, 2026

Copy link
Copy Markdown
Member

Pull requests targeting this branch never run CodeQL: the chart/
release-branch patterns are missing from this branch's CodeQL trigger filters,
so the security analysis that gates main is silently absent here while the
rest of the test suite reports normally. Verified on the most recent merged PR
into this branch — a full green test suite, zero CodeQL checks.

The test pipeline itself is unaffected and already runs, so ci-amd-arm.yml is
deliberately left alone. Only the CodeQL filters change.

GitHub resolves trigger filters from the workflow files on the target branch,
so the equivalent fix on main (#69626) is a no-op here and has to land on the
release branch itself.

related: #69626


Was generative AI tooling used to co-author this PR?
  • Yes — Claude Code (Opus 5)

Generated-by: Claude Code (Opus 5) following the guidelines

Pull requests targeting this branch never ran CodeQL: the chart
release-branch patterns were missing from the CodeQL trigger filters, so
the security analysis that gates main was silently absent here while the
rest of the test suite reported normally.

GitHub resolves trigger filters from the workflow files on the target
branch, so the equivalent fix on main (apache#69626) does not apply here and
has to land on the release branch itself.
@Lee-W
Lee-W marked this pull request as ready for review July 31, 2026 03:32
@Lee-W
Lee-W merged commit 4062591 into apache:chart/v1-2x-test Jul 31, 2026
246 checks passed
@Lee-W
Lee-W deleted the codeql-trigger-chart-v1-2x-test branch July 31, 2026 06:38
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.

2 participants