-
Notifications
You must be signed in to change notification settings - Fork 0
Add CodeQL analysis workflow configuration #102
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
base: master
Are you sure you want to change the base?
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,107 @@ | ||||||||||||||||||||||||||||||||||||||||||
| # 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: '19 4 * * 4' | ||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||
| 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: actions | ||||||||||||||||||||||||||||||||||||||||||
| build-mode: none | ||||||||||||||||||||||||||||||||||||||||||
| - language: javascript-typescript | ||||||||||||||||||||||||||||||||||||||||||
| build-mode: none | ||||||||||||||||||||||||||||||||||||||||||
| - language: python | ||||||||||||||||||||||||||||||||||||||||||
| build-mode: none | ||||||||||||||||||||||||||||||||||||||||||
| - language: rust | ||||||||||||||||||||||||||||||||||||||||||
| build-mode: none | ||||||||||||||||||||||||||||||||||||||||||
| - language: swift | ||||||||||||||||||||||||||||||||||||||||||
| build-mode: autobuild | ||||||||||||||||||||||||||||||||||||||||||
|
Comment on lines
+46
to
+55
|
||||||||||||||||||||||||||||||||||||||||||
| - language: actions | |
| build-mode: none | |
| - language: javascript-typescript | |
| build-mode: none | |
| - language: python | |
| build-mode: none | |
| - language: rust | |
| build-mode: none | |
| - language: swift | |
| build-mode: autobuild | |
| - language: actions | |
| build-mode: none | |
| - language: javascript-typescript | |
| build-mode: none | |
| - language: python | |
| build-mode: none | |
| - language: rust | |
| build-mode: none | |
| - language: swift | |
| build-mode: autobuild |
Copilot
AI
Apr 6, 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.
The workflow matrix includes Swift and conditionally switches to a macOS runner for it, but there doesn’t appear to be any Swift code in this repo. Keeping Swift here adds a macOS job (cost/time) and potential failures; consider removing Swift from the matrix and simplifying runs-on accordingly.
Copilot
AI
Apr 6, 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.
Other workflows in this repo use actions/checkout@v6; this workflow uses actions/checkout@v4. For consistency (and to stay current with the rest of the repo’s CI), consider updating to the same major version used elsewhere.
| uses: actions/checkout@v4 | |
| uses: actions/checkout@v6 |
Copilot
AI
Apr 6, 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: is followed by step entries (- name:) at the same indentation level. In GitHub Actions YAML, the list items must be indented under steps:; otherwise the workflow will be invalid.
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.
This workflow only listens on
master, but the repository currently has nomasterbranch, so CodeQL will not run for normalpush/pull_requestactivity and findings are deferred to the weekly cron run. That leaves a large gap where insecure changes can merge without code scanning; point these branch filters at the real default branch (or remove the filter) to restore per-PR/per-push coverage.Useful? React with 👍 / 👎.