Skip to content

Commit

Permalink
chore: switch to c8 instead of nyc
Browse files Browse the repository at this point in the history
I wanted to cut another RC to potentially close
* #1420
* #1398
* #1397

but the release is erroring at the test because of the memory limit
so I decided to use c8 instead of nyc because when trying to remove nyc
I concluded that the memory error comes from it and its last release was
4 years ago. It works as a drop-in replacement.

https://www.npmjs.com/package/c8
  • Loading branch information
mstruebing committed Dec 15, 2023
1 parent 7d69434 commit 665608e
Show file tree
Hide file tree
Showing 3 changed files with 253 additions and 2,116 deletions.
2 changes: 0 additions & 2 deletions .github/workflows/test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,5 @@ jobs:
- run: node version-check.js
- run: npm ci
- run: npm test
env:
NODE_OPTIONS: --max-old-space-size=8192
- run: npm run lint
- run: npm audit --audit-level=critical

0 comments on commit 665608e

Please sign in to comment.