-
Notifications
You must be signed in to change notification settings - Fork 0
Add CodeQL analysis workflow configuration #3
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change | ||||||||
|---|---|---|---|---|---|---|---|---|---|---|
| @@ -0,0 +1,99 @@ | ||||||||||
| # For most projects, this workflow file will not need changing; you simply need | ||||||||||
| # to commit it to your repository. | ||||||||||
| # | ||||||||||
| # You may wish to alter this file to override the set of languages analyzed, | ||||||||||
| # or to provide custom queries or build logic. | ||||||||||
| # | ||||||||||
| # ******** NOTE ******** | ||||||||||
| # We have attempted to detect the languages in your repository. Please check | ||||||||||
| # the `language` matrix defined below to confirm you have the correct set of | ||||||||||
| # supported CodeQL languages. | ||||||||||
| # | ||||||||||
| name: "CodeQL Advanced" | ||||||||||
|
|
||||||||||
| on: | ||||||||||
| push: | ||||||||||
| branches: [ "master" ] | ||||||||||
| pull_request: | ||||||||||
| branches: [ "master" ] | ||||||||||
| schedule: | ||||||||||
| - cron: '23 6 * * 5' | ||||||||||
|
|
||||||||||
| jobs: | ||||||||||
| analyze: | ||||||||||
| name: Analyze (${{ matrix.language }}) | ||||||||||
| # Runner size impacts CodeQL analysis time. To learn more, please see: | ||||||||||
| # - https://gh.io/recommended-hardware-resources-for-running-codeql | ||||||||||
| # - https://gh.io/supported-runners-and-hardware-resources | ||||||||||
| # - https://gh.io/using-larger-runners (GitHub.com only) | ||||||||||
| # Consider using larger runners or machines with greater resources for possible analysis time improvements. | ||||||||||
| runs-on: ${{ (matrix.language == 'swift' && 'macos-latest') || 'ubuntu-latest' }} | ||||||||||
| permissions: | ||||||||||
| # required for all workflows | ||||||||||
| security-events: write | ||||||||||
|
|
||||||||||
| # required to fetch internal or private CodeQL packs | ||||||||||
| packages: read | ||||||||||
|
|
||||||||||
| # only required for workflows in private repositories | ||||||||||
| actions: read | ||||||||||
| contents: read | ||||||||||
|
|
||||||||||
| strategy: | ||||||||||
| fail-fast: false | ||||||||||
| matrix: | ||||||||||
| include: | ||||||||||
| - language: javascript-typescript | ||||||||||
| build-mode: none | ||||||||||
|
Comment on lines
+46
to
+47
|
||||||||||
| - language: javascript-typescript | |
| build-mode: none | |
| - language: javascript-typescript | |
| build-mode: none |
Copilot
AI
Mar 12, 2026
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Minor grammar in comment: "values keywords" reads incorrectly; consider changing to just "values" or "keywords".
| # CodeQL supports the following values keywords for 'language': 'actions', 'c-cpp', 'csharp', 'go', 'java-kotlin', 'javascript-typescript', 'python', 'ruby', 'rust', 'swift' | |
| # CodeQL supports the following values for 'language': 'actions', 'c-cpp', 'csharp', 'go', 'java-kotlin', 'javascript-typescript', 'python', 'ruby', 'rust', 'swift' |
Copilot
AI
Mar 12, 2026
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
steps: must contain a YAML sequence indented beneath it. The - name: ... entries are currently aligned with steps: which will break workflow parsing; indent the step items under steps:.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The workflow filters both
pushandpull_requestevents tomaster, but this repository’s active branch ismain, so CodeQL will not run on normal commits/PRs and will only execute on the weekly schedule. That leaves new vulnerabilities or query regressions undetected during everyday development until the cron run; update the branch filters tomain(or include both branch names if needed).Useful? React with 👍 / 👎.