Skip to content

Add CodeQL analysis workflow configuration#3654

Merged
ahpook merged 2 commits intomainfrom
ahpook/add-codeql-config
Apr 29, 2026
Merged

Add CodeQL analysis workflow configuration#3654
ahpook merged 2 commits intomainfrom
ahpook/add-codeql-config

Conversation

@ahpook
Copy link
Copy Markdown
Contributor

@ahpook ahpook commented Apr 29, 2026

The default configuration is reporting a syntax error on the file, but the file is 'invisible' because we're running in default CodeQL configuration. So I'm switching to Advanced mode to get an concrete file to operate on.

  • Have you followed the contributing guidelines?
  • Have you explained what your changes do, and why they add value to the Guides?

Please note: we will close your PR without comment if you do not check the boxes above and provide ALL requested information.


The default configuration is reporting a syntax error on the file, but the file is 'invisible' because we're running in default CodeQL configuration. So I'm switching to Advanced mode to get an concrete file to operate on.
Copilot AI review requested due to automatic review settings April 29, 2026 21:02
@ahpook ahpook requested a review from a team as a code owner April 29, 2026 21:02
@ahpook ahpook temporarily deployed to Pages Preview April 29, 2026 21:02 — with GitHub Actions Inactive
@ahpook ahpook temporarily deployed to Pages Preview April 29, 2026 21:03 — with GitHub Actions Inactive
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds an explicit CodeQL Advanced workflow so the repository’s code scanning uses a concrete, configurable CodeQL setup (instead of the default configuration where the problematic file was not directly addressable).

Changes:

  • Introduces a new .github/workflows/codeql.yml workflow using CodeQL Advanced.
  • Configures a language matrix (actions, javascript-typescript, ruby) and scheduled runs.
Show a summary per file
File Description
.github/workflows/codeql.yml Adds CodeQL Advanced workflow configuration for multi-language scanning.

Copilot's findings

Tip

Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comments suppressed due to low confidence (2)

.github/workflows/codeql.yml:62

  • steps: is followed by list items that are not indented under it (lines 60-62). This is invalid YAML/workflow syntax; the - name: entries need to be indented beneath steps: so GitHub Actions recognizes them as the job's step list.
    steps:
    - name: Checkout repository
      uses: actions/checkout@v4

.github/workflows/codeql.yml:73

  • Other workflows in this repo pin GitHub Actions to specific release versions (e.g., actions/checkout@v6.0.2, actions/stale@v10.2.0) rather than major tags. To stay consistent (and reduce supply-chain risk), consider pinning actions/checkout and github/codeql-action/* to specific versions here too instead of @v4.
    - name: Checkout repository
      uses: actions/checkout@v4

    # Add any setup steps before running the `github/codeql-action/init` action.
    # This includes steps like installing compilers or runtimes (`actions/setup-node`
    # or others). This is typically only required for manual builds.
    # - name: Setup runtime (example)
    #   uses: actions/setup-example@v1

    # Initializes the CodeQL tools for scanning.
    - name: Initialize CodeQL
      uses: github/codeql-action/init@v4
      with:
  • Files reviewed: 1/1 changed files
  • Comments generated: 2

Comment thread .github/workflows/codeql.yml
Comment thread .github/workflows/codeql.yml
- trim comments and run yaml validator on codeql.yaml
- change stale reference to `master` branch in tests to `main`
@ahpook ahpook temporarily deployed to Pages Preview April 29, 2026 21:44 — with GitHub Actions Inactive
@ahpook ahpook merged commit 78e719e into main Apr 29, 2026
7 checks passed
@ahpook ahpook deleted the ahpook/add-codeql-config branch April 29, 2026 21:49
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