Skip to content

ci: add CodeQL workflow#558

Open
cameri wants to merge 2 commits intomainfrom
ci/add-codeql-workflow
Open

ci: add CodeQL workflow#558
cameri wants to merge 2 commits intomainfrom
ci/add-codeql-workflow

Conversation

@cameri
Copy link
Copy Markdown
Owner

@cameri cameri commented Apr 22, 2026

Description

Related Issue

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):

Types of changes

  • Non-functional change (docs, style, minor refactor)
  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my code changes.
  • I added a changeset, or this is docs-only and I added an empty changeset.
  • All new and existing tests passed.

Copilot AI review requested due to automatic review settings April 22, 2026 00:12
@cameri cameri self-assigned this Apr 22, 2026
@github-advanced-security
Copy link
Copy Markdown

You are seeing this message because GitHub Code Scanning has recently been set up for this repository, or this pull request contains the workflow file for the Code Scanning tool.

What Enabling Code Scanning Means:

  • The 'Security' tab will display more code scanning analysis results (e.g., for the default branch).
  • Depending on your configuration and choice of analysis tool, future pull requests will be annotated with code scanning analysis results.
  • You will be able to see the analysis results for the pull request's branch on this overview once the scans have completed and the checks have passed.

For more information about GitHub Code Scanning, check out the documentation.

Copy link
Copy Markdown

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 a GitHub Actions workflow to run CodeQL code scanning on pushes/PRs to main and on a weekly schedule, enabling automated security/static analysis for the repo.

Changes:

  • Introduces a new .github/workflows/codeql.yml workflow for CodeQL scanning.
  • Configures a language matrix for actions and javascript-typescript.
  • Runs CodeQL init/analyze steps (with a placeholder manual build step gated on build-mode: manual).

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

Comment on lines +47 to +50
- language: actions
build-mode: none
- language: javascript-typescript
build-mode: none
Copy link

Copilot AI Apr 22, 2026

Choose a reason for hiding this comment

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

The matrix.include list items appear mis-indented relative to include: and their nested keys (build-mode). This can make the workflow YAML invalid or parse differently than intended. Re-indent include entries so the - language lines are consistently nested under include: and build-mode aligns as a sibling key within each list item (using standard 2-space YAML indentation).

Suggested change
- language: actions
build-mode: none
- language: javascript-typescript
build-mode: none
- language: actions
build-mode: none
- language: javascript-typescript
build-mode: none

Copilot uses AI. Check for mistakes.
Comment on lines +59 to +62
steps:
- name: Checkout repository
uses: actions/checkout@v4

Copy link

Copilot AI Apr 22, 2026

Choose a reason for hiding this comment

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

steps: is defined, but the step list entries (- name: ...) are not indented under it. As written, this makes the workflow YAML invalid and GitHub Actions will fail to load the workflow. Indent all step items and their fields so they are nested under steps:.

Copilot uses AI. Check for mistakes.
@coveralls
Copy link
Copy Markdown
Collaborator

coveralls commented Apr 22, 2026

Coverage Status

coverage: 74.929%. remained the same — ci/add-codeql-workflow into main

@changeset-bot
Copy link
Copy Markdown

changeset-bot Bot commented Apr 22, 2026

⚠️ No Changeset found

Latest commit: 7765b57

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

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.

4 participants