ci: remove custom CodeQL workflow — GitHub Default setup is already enabled#34
Merged
Conversation
GitHub's Default setup detected two missing configurations on main: - /language:rust - /language:actions This workflow runs CodeQL analysis for both languages on every push to main, every pull request targeting main, and on a weekly schedule (Monday 06:15 UTC). The category values match the slugs that GitHub expects, resolving the 'configurations not found' warning. Co-authored-by: Andy Babic <ababic@users.noreply.github.com>
The custom codeql.yml conflicted with GitHub's Default setup in two ways: - Rust does not support build-mode: manual (only 'none' is supported) - SARIF uploads from advanced configurations are rejected when Default setup is already enabled for the repository GitHub's Default setup already handles /language:rust and /language:actions scanning automatically. The 'configurations not found' warning only appeared because Default setup had not yet run on the PR branch; once merged to main it runs normally. The custom workflow is not needed and must be removed. Co-authored-by: Andy Babic <ababic@users.noreply.github.com>
d34d6b6 to
eedc913
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Problem
GitHub's built-in security scanner raised the following warning on pull requests:
Root Cause
The warning is produced by GitHub's Default setup, which was already enabled for this repository and already scanning for both
language:rustandlanguage:actions. The warning only appeared on PRs because Default setup had not yet been triggered on those PR branches — it runs automatically once a PR is opened againstmain.A first attempt added a custom
codeql.ymlworkflow, but that conflicted with Default setup in two ways (confirmed by CI logs):build-mode: manualis not supported — CodeQL's Rust extractor only supportsbuild-mode: none.CodeQL analyses from advanced configurations cannot be processed when the default setup is enabled. GitHub explicitly blocks custom CodeQL workflows when Default setup is active on the same repository.Fix
Removing the custom
codeql.ymlentirely. GitHub's Default setup already covers both languages. No additional configuration is needed — the original warning will stop appearing once this PR is merged and Default setup has a baseline onmainto compare against.Slack Thread