Skip to content

Align Node.js CI matrix with project runtime requirements - #1

Closed
04harithecoder with Copilot wants to merge 2 commits into
mainfrom
copilot/fix-failing-github-actions-job
Closed

04harithecoder with Copilot wants to merge 2 commits into
mainfrom
copilot/fix-failing-github-actions-job

Conversation

Copilot AI commented Aug 20, 2026 •

Copy link
Copy Markdown

build (18.x) was failing in Actions because the current Vite/Rolldown toolchain requires Node 20+, while the workflow still included Node 18. This PR updates CI to run only on supported Node versions and avoids false-negative test step failures when no test script is defined.

  • CI runtime compatibility

    • Remove 18.x from the Node matrix in .github/workflows/node.js.yml
    • Keep CI coverage on supported LTS versions (20.x, 22.x)
  • Test step resilience

    • Change npm test to npm test --if-present so repositories without a test script do not fail the pipeline by default
strategy:
  matrix:
    node-version: [20.x, 22.x]

steps:
  - run: npm ci
  - run: npm run build --if-present
  - run: npm test --if-present

Co-authored-by: 04harithecoder <201968851+04harithecoder@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix failing GitHub Actions job build (18.x) Align Node.js CI matrix with project runtime requirements Aug 21, 2026
Copilot AI requested a review from 04harithecoder August 21, 2026 00:00
@04harithecoder
04harithecoder deleted the copilot/fix-failing-github-actions-job branch September 20, 2026 14:35
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