Skip to content

Add advanced CodeQL workflow steps#941

Merged
decyjphr merged 1 commit intomain-enterprisefrom
decyjphr-patch-1
Mar 3, 2026
Merged

Add advanced CodeQL workflow steps#941
decyjphr merged 1 commit intomain-enterprisefrom
decyjphr-patch-1

Conversation

@decyjphr
Copy link
Collaborator

@decyjphr decyjphr commented Mar 3, 2026

New CodeQL workflow setup:

  • Added .github/workflows/advanced-codeql.yml to enable advanced CodeQL security scanning on main-enterprise branch and pull requests, with support for multiple languages (actions, javascript-typescript) and customizable build modes.

Copilot AI review requested due to automatic review settings March 3, 2026 12:47
@decyjphr decyjphr merged commit eef9762 into main-enterprise Mar 3, 2026
9 of 11 checks passed
@decyjphr decyjphr deleted the decyjphr-patch-1 branch March 3, 2026 12:49
Copy link
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 a new GitHub Actions workflow to run “advanced” CodeQL code scanning on the main-enterprise branch and related pull requests, targeting multiple languages used in this repo.

Changes:

  • Added .github/workflows/advanced-codeql.yml with a matrix for CodeQL analysis across actions and javascript-typescript.
  • Configured triggers for workflow_dispatch, push, and pull_request on main-enterprise.
Comments suppressed due to low confidence (1)

.github/workflows/advanced-codeql.yml:63

  • steps: list entries are not indented beneath steps:. YAML requires the - name: items to be nested under the steps key; otherwise this workflow is likely invalid and won't run.
    steps:
    - name: 'Checkout repository'
      uses: actions/checkout@v4
    # Initializes the CodeQL tools for scanning.
    - name: Initialize CodeQL
      uses: github/codeql-action/init@v4

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

# your codebase is analyzed, see https://docs.github.com/en/code-security/code-scanning/creating-an-advanced-setup-for-code-scanning/codeql-code-scanning-for-compiled-languages
steps:
- name: 'Checkout repository'
uses: actions/checkout@v4
Copy link

Copilot AI Mar 3, 2026

Choose a reason for hiding this comment

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

This repo’s other workflows use actions/checkout@v6 (e.g., .github/workflows/node-ci.yml). Consider updating this workflow to the same major version to keep action versions consistent across the repo.

Suggested change
uses: actions/checkout@v4
uses: actions/checkout@v6

Copilot uses AI. Check for mistakes.
Comment on lines +44 to +49
matrix:
include:
- language: actions
build-mode: none
- language: javascript-typescript
build-mode: none
Copy link

Copilot AI Mar 3, 2026

Choose a reason for hiding this comment

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

matrix.include items are not indented under include:. In YAML, the sequence entries must be nested (e.g., include: then items indented beneath). As written, the workflow file may fail to parse, preventing CodeQL from running.

This issue also appears on line 58 of the same file.

Copilot uses AI. Check for mistakes.
name: "CodeQL Advanced"

on:
workflow_dispatch:
Copy link

Copilot AI Mar 3, 2026

Choose a reason for hiding this comment

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

There is trailing whitespace after workflow_dispatch:. Consider removing it to avoid lint noise and keep formatting consistent with other workflows.

Suggested change
workflow_dispatch:
workflow_dispatch:

Copilot uses AI. Check for mistakes.
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