Skip to content

ci: migrate CodeQL from default to advanced setup - #11

Merged
NullSablex merged 2 commits into
masterfrom
ci/codeql-advanced-setup
Aug 30, 2026
Merged

ci: migrate CodeQL from default to advanced setup#11
NullSablex merged 2 commits into
masterfrom
ci/codeql-advanced-setup

Conversation

@NullSablex

Copy link
Copy Markdown
Owner

The CodeQL default setup only analysed a pull request when it touched files relevant to the configured languages. A PR changing only docs or a dependency manifest produced no analysis at all, while master still carried one per language — so the code_scanning branch rule could not diff the two sides and reported:

Code scanning cannot determine the alerts introduced by this pull request, because 2 configurations present on refs/heads/master were not found: Default setup — /language:actions, /language:rust

This adds .github/workflows/codeql.yml (advanced setup) with no path filter, so both configurations exist on every PR. Categories stay /language:actions and /language:rust to match what is already recorded on master, keeping the diff continuous.

The default setup has already been disabled via the API — the two are mutually exclusive and codeql-action/init refuses to run while it is on.

The default setup only analysed a pull request when it touched files
relevant to the configured languages. A PR changing only docs or a
dependency manifest produced no analysis, while master still carried one
per language, so the code_scanning branch rule could not diff the two
sides and reported '2 configurations not found'.

Add a codeql.yml with no path filter so both configurations exist on
every PR, keeping the /language:actions and /language:rust categories
recorded on master.
@NullSablex
NullSablex merged commit 2e2de81 into master Aug 30, 2026
7 checks passed
@NullSablex
NullSablex deleted the ci/codeql-advanced-setup branch August 30, 2026 21:06
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