Skip to content

ci: restrict the Node test matrix to LTS majors by default - #11

Merged
Shinrai merged 1 commit into
nextfrom
ci/lts-only-matrix
Jul 19, 2026
Merged

ci: restrict the Node test matrix to LTS majors by default#11
Shinrai merged 1 commit into
nextfrom
ci/lts-only-matrix

Conversation

@cldmv-bot

@cldmv-bot cldmv-bot Bot commented Jul 19, 2026

Copy link
Copy Markdown
Contributor

🚀 What's Changed

💥 Breaking Changes

No breaking changes

✨ Features

No new features

🐛 Bug Fixes

No bug fixes

📦 Dependencies

No dependency updates

🔧 Other Changes

  • ci: restrict the Node test matrix to LTS majors by default (43ebcfa)
👥 Contributors

The matrix ran full (20/21/22/lts) on every event except direct pushes to
master (`lts_only_matrix` was off-by-default). Node 21 is a non-LTS interim
release the native-binding test toolchain (vitest 4 / rolldown / vite 8)
excludes via `engines` (`^20.19.0 || >=22.12.0`), so its matrix leg dies at
startup with "Cannot find native binding" — not a real per-version
regression. This is what turned CI red on next: the release PR #3 runs as a
pull_request event (ref refs/pull/3/merge), so it never matched the
master-only LTS toggle and got the full matrix incl. the broken 21 leg.

Default `lts_only_matrix` on for all events (workflow_dispatch can still opt
out with false). Matrix is now 20/22/lts — the broken 21 leg is gone, per
the LTS-only-majors CI policy.
@cldmv-bot cldmv-bot Bot added the ! ci → next v4 flow: ci contributor PR targeting the next integration branch label Jul 19, 2026
@Shinrai
Shinrai merged commit 8a089f6 into next Jul 19, 2026
18 checks passed
@cldmv-bot
cldmv-bot Bot deleted the ci/lts-only-matrix branch July 19, 2026 06:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

! ci → next v4 flow: ci contributor PR targeting the next integration branch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant