Skip to content

[ci-maintainer] fix: activate CodeQL SAST workflow#6278

Closed
clubanderson wants to merge 1 commit into
mainfrom
ci/fix-docs-codeql-enable
Closed

[ci-maintainer] fix: activate CodeQL SAST workflow#6278
clubanderson wants to merge 1 commit into
mainfrom
ci/fix-docs-codeql-enable

Conversation

@clubanderson

Copy link
Copy Markdown
Contributor

CI Fix

PR #6274 created .github/workflows/codeql.yml.disabled instead of
.github/workflows/codeql.yml. GitHub Actions ignores .disabled files,
so SAST was never running despite the workflow being present.

This PR creates the correctly-named active workflow file with the same
content, enabling CodeQL SAST on push to main and all pull requests.

Fixes #6275


Filed by ci-maintainer agent (ACMM L6 — full mode)

…bled)

Fixes #6275

Signed-off-by: ci-maintainer <ci-maintainer@kubestellar.io>

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings July 10, 2026 06:16
@clubanderson
clubanderson requested a review from KPRoche as a code owner July 10, 2026 06:16
@kubestellar-prow kubestellar-prow Bot added the dco-signoff: yes Indicates the PR's author has signed the DCO. label Jul 10, 2026
@kubestellar-prow

Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please assign kproche for approval. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@clubanderson clubanderson added ci and removed dco-signoff: yes Indicates the PR's author has signed the DCO. labels Jul 10, 2026
@netlify

netlify Bot commented Jul 10, 2026

Copy link
Copy Markdown

Deploy Preview for kubestellar-docs ready!

Name Link
🔨 Latest commit b319621
🔍 Latest deploy log https://app.netlify.com/projects/kubestellar-docs/deploys/6a508e59e5d7970008042609
😎 Deploy Preview https://deploy-preview-6278--kubestellar-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@kubestellar-prow kubestellar-prow Bot added the size/M Denotes a PR that changes 30-99 lines, ignoring generated files. label Jul 10, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR activates CodeQL SAST by adding a properly named GitHub Actions workflow file (.github/workflows/codeql.yml), addressing the prior misnaming (.yml.disabled) that prevented the workflow from running.

Changes:

  • Adds an enabled CodeQL workflow that runs on push to main, pull_request targeting main, and a weekly schedule.
  • Configures CodeQL to analyze javascript-typescript with the security-and-quality query suite.

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

Comment on lines +18 to +20
permissions:
contents: read
security-events: write
Comment on lines +22 to +23
- name: Checkout repository
uses: actions/checkout@df4cb1c069e1874edd31b4311f1884172cec0e10 # v6.0.3
@clubanderson

Copy link
Copy Markdown
Contributor Author

Closing — .github/workflows/codeql.yml already exists on main with the same content (plus workflow_dispatch trigger and actions language). This PR is redundant and has merge conflicts because the file was already added via another path.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci ci-cd size/M Denotes a PR that changes 30-99 lines, ignoring generated files. yaml

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[sec-check] SAST not running on all commits — Scorecard alert #71 persists (re-filed, prior #6271 closed)

2 participants