Skip to content

Update Node.js version to 22.x and remove matrix testing#21

Merged
pheuberger merged 1 commit into
mainfrom
claude/review-node-test-versions-LGWh8
Feb 5, 2026
Merged

Update Node.js version to 22.x and remove matrix testing#21
pheuberger merged 1 commit into
mainfrom
claude/review-node-test-versions-LGWh8

Conversation

@pheuberger
Copy link
Copy Markdown
Owner

Summary

This PR updates the CI/CD pipeline to use Node.js 22.x as the single target version and removes the matrix strategy that previously tested against multiple Node.js versions (18.x and 20.x).

Key Changes

  • Removed Node.js version matrix: Eliminated the strategy matrix that tested against Node.js 18.x and 20.x
  • Updated to Node.js 22.x: Changed all Node.js setup steps to use version 22.x across all jobs (test, lint, and type-check)
  • Simplified conditional logic: Removed all if: matrix.node-version == '20.x' conditions from test, coverage, and reporting steps since there's now only one version
  • Streamlined coverage reporting: Coverage tests, uploads, and PR comments now run unconditionally (except PR comment which still checks github.event_name == 'pull_request')

Implementation Details

  • The test job no longer uses a matrix strategy, reducing CI run time by running tests only once per commit
  • All coverage-related steps (testing with coverage, uploading artifacts, commenting on PRs, and generating summaries) now execute in every test run
  • The lint and type-check jobs were also updated to use Node.js 22.x for consistency
  • This change assumes Node.js 22.x is the minimum supported version going forward

https://claude.ai/code/session_01DXd2UwS1bSM3mGBHiS3Nrc

- Remove Node version matrix (was testing 18.x and 20.x)
- Update all jobs to use Node 22.x (current Active LTS)
- Remove redundant matrix conditions from coverage steps

Node 18 reached EOL in April 2025, and multi-version testing
provides no value for a browser application where Node is only
used for build tooling.

https://claude.ai/code/session_01DXd2UwS1bSM3mGBHiS3Nrc
@netlify
Copy link
Copy Markdown

netlify Bot commented Feb 5, 2026

Deploy Preview for hypermarkk ready!

Name Link
🔨 Latest commit b2a026e
🔍 Latest deploy log https://app.netlify.com/projects/hypermarkk/deploys/698437bc4655c90008a94bc8
😎 Deploy Preview https://deploy-preview-21--hypermarkk.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.

@pheuberger pheuberger merged commit 7c18eb2 into main Feb 5, 2026
7 checks passed
@pheuberger pheuberger deleted the claude/review-node-test-versions-LGWh8 branch February 5, 2026 07:51
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