Skip to content

chore: support minimum node 20, use native node test runner #230

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Draft
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

filmaj
Copy link
Member

@filmaj filmaj commented Apr 7, 2025

playing around to see how hard this will be. seems to work pretty well! in prep for node 18 being EOL'ed in April.

  • min node 20+; CI tests node 20, 22 and 24
  • use native node --test runner, remove tape and any testing related dependencies

this approach is much faster on my macbook w/ node v20:

$ time npm run coverage
...
npm run coverage  1.40s user 0.27s system 239% cpu 0.696 total

compared to main:

➜ time npm run coverage
...
npm run coverage  4.11s user 0.75s system 87% cpu 5.526 total

@filmaj filmaj self-assigned this Apr 7, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

1 participant