Skip to content

ci: add explicit CodeQL workflow for PR and push#116

Open
novatechflow wants to merge 3 commits intoKafScale:mainfrom
novatechflow:novatechflow/codeql-pr-scans
Open

ci: add explicit CodeQL workflow for PR and push#116
novatechflow wants to merge 3 commits intoKafScale:mainfrom
novatechflow:novatechflow/codeql-pr-scans

Conversation

@novatechflow
Copy link
Collaborator

Summary

  • add explicit .github/workflows/codeql.yml
  • run CodeQL on pull_request to main and push to main
  • include workflow_dispatch for manual reruns

Why

Default CodeQL setup was running on merge/main but not on current fork-based PR heads. This workflow makes scans deterministic for PR commits and main pushes.

uses: github/codeql-action/init@v4
with:
languages: ${{ matrix.language }}
query-filters: security-extended,security-and-quality
Copy link
Collaborator

@klaudworks klaudworks Mar 1, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The input parameter is called queries: https://github.com/github/codeql-action/blob/89a39a4e59826350b863aa6b6252a07ad50cf83e/init/action.yml

  queries:
    description: Comma-separated list of additional queries to run. By default, this overrides the same setting in a configuration file; prefix with "+" to use both sets of queries.
    required: false
Suggested change
query-filters: security-extended,security-and-quality
queries: security-extended,security-and-quality

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think query-filters is silently skipped

@klaudworks klaudworks self-requested a review March 1, 2026 08:24
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.

2 participants