Skip to content

Comments

Fix undefined matrix variables in CodeQL workflow#10

Merged
jusuchin85 merged 3 commits intojusuchin85-patch-2from
copilot/sub-pr-9
Feb 6, 2026
Merged

Fix undefined matrix variables in CodeQL workflow#10
jusuchin85 merged 3 commits intojusuchin85-patch-2from
copilot/sub-pr-9

Conversation

Copy link

Copilot AI commented Feb 6, 2026

The CodeQL workflow referenced matrix.language and matrix.build-mode throughout but never defined them in the matrix configuration, causing the workflow to fail or skip steps.

Changes:

  • Added language: javascript-typescript and build-mode: none to matrix.include
  • Added inline comment explaining the configuration for this HTML/JavaScript project

Affected references:

# Job name, runner selection, init action, conditional steps, and analysis categorization
name: Analyze (${{ matrix.language }})
runs-on: ${{ (matrix.language == 'swift' && 'macos-latest') || 'ubuntu-latest' }}
languages: ${{ matrix.language }}
build-mode: ${{ matrix.build-mode }}
if: matrix.build-mode == 'manual'
category: "/language:${{matrix.language}}"

All six references now resolve to defined values.


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Copilot AI and others added 2 commits February 6, 2026 00:09
Co-authored-by: jusuchin85 <1155821+jusuchin85@users.noreply.github.com>
Co-authored-by: jusuchin85 <1155821+jusuchin85@users.noreply.github.com>
Copilot AI changed the title [WIP] Update CodeQL analysis workflow configuration based on feedback Fix undefined matrix variables in CodeQL workflow Feb 6, 2026
Copilot AI requested a review from jusuchin85 February 6, 2026 00:11
@jusuchin85 jusuchin85 marked this pull request as ready for review February 6, 2026 00:12
@jusuchin85 jusuchin85 merged commit b232457 into jusuchin85-patch-2 Feb 6, 2026
@jusuchin85 jusuchin85 deleted the copilot/sub-pr-9 branch February 6, 2026 00:12
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