Skip to content

workflows: drop old node, add 24.x, only lint once#410

Open
MaxLeiter wants to merge 2 commits intomasterfrom
max/04-10-cleanup
Open

workflows: drop old node, add 24.x, only lint once#410
MaxLeiter wants to merge 2 commits intomasterfrom
max/04-10-cleanup

Conversation

@MaxLeiter
Copy link
Copy Markdown
Collaborator

  • Dropped Node 16.x and 18.x, added 24.x
  • Split into two jobs: lint (runs once on Node 22) and test. before we were running lint on every version
  • Test job runs mocha + nyc directly nstead of also running lint

Copy link
Copy Markdown

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

Updates the CI test workflow to better align with the project’s supported Node.js versions while reducing redundant work in GitHub Actions.

Changes:

  • Dropped Node.js 16.x/18.x from the test matrix and added 24.x.
  • Split CI into a dedicated lint job (runs once on Node 22.x) and a test matrix job.
  • Updated the test job to run yarn coverage (nyc+mocha) instead of yarn test (which also runs lint).

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

Comment thread .github/workflows/tests.yml
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
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