Skip to content

Audit and fix GitHub Actions workflows#14

Open
Copilot wants to merge 2 commits intomainfrom
copilot/audit-github-actions-setup
Open

Audit and fix GitHub Actions workflows#14
Copilot wants to merge 2 commits intomainfrom
copilot/audit-github-actions-setup

Conversation

Copy link
Contributor

Copilot AI commented Mar 18, 2026

Five workflows existed; three had real problems. Removes matrix noise, eliminates a single-item strategy matrix, and fixes a recurring Scorecards failure that was red on every main push.

Changes

  • ci.yml — Drop Node 20/22 matrix. engines.node >= 20 and action.yml both pin node20; a second version adds runner cost with no signal.
  • codeql.yml — Remove single-item language matrix. Replaced ${{ matrix.language }} with the literal javascript-typescript directly.
  • scorecards.yml — Flip publish_results: truefalse. The Scorecards webapp was returning HTTP 400 ("workflow verification failed") on every push to main. Analysis and SARIF upload to code scanning still run.

Kept unchanged

  • dependency-review.yml — Clean and correct as-is.
  • action-bundle-check.yml — Valid: guards that the committed apps/action/dist/index.js bundle stays in sync with source.

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Co-authored-by: chrismaz11 <24700273+chrismaz11@users.noreply.github.com>
Copilot AI changed the title [WIP] Audit and fix GitHub Actions workflows Audit and fix GitHub Actions workflows Mar 18, 2026
Copilot AI requested a review from chrismaz11 March 18, 2026 03:44
@chrismaz11 chrismaz11 marked this pull request as ready for review March 18, 2026 04:01
Copilot AI review requested due to automatic review settings March 18, 2026 04:01
@github-advanced-security
Copy link

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

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

This PR audits and simplifies GitHub Actions workflows by removing redundant matrices and addressing a persistent Scorecards publishing failure while keeping Code Scanning (SARIF) uploads intact.

Changes:

  • Simplify CI by removing the Node version matrix and pinning the workflow to Node 20.
  • Remove a single-item language matrix from the CodeQL workflow and inline the language value.
  • Disable Scorecards “publish_results” to avoid the recurring publish/verification failure while still uploading SARIF to code scanning.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File Description
.github/workflows/ci.yml Removes Node matrix and runs validation on Node 20 only.
.github/workflows/codeql.yml Deletes single-item matrix and hardcodes javascript-typescript in CodeQL init.
.github/workflows/scorecards.yml Sets publish_results: false while keeping SARIF upload step.

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

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.

3 participants