-
-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
fix(coverage): v8 to ignore type-only files #5328
fix(coverage): v8 to ignore type-only files #5328
Conversation
✅ Deploy Preview for fastidious-cascaron-4ded94 ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
/ecosystem-ci run |
📝 Ran ecosystem CI: Open
|
Nuxt's failure looks like real issue. It's passing fine on |
10265d5
to
ef21bca
Compare
It was a file that failed in |
ef21bca
to
2c3a910
Compare
Confirmed this fixed a whole bunch of |
Description
coverage.all
typescript files with no executable code are colored red in the coverage report #3605coverage.json
cover this feature. If the fix is removed, the newtypes.ts
shows up in snapshot and it fails.Please don't delete this checklist! Before submitting the PR, please make sure you do the following:
pnpm-lock.yaml
unless you introduce a new test example.Tests
pnpm test:ci
.Documentation
pnpm run docs
command.Changesets
feat:
,fix:
,perf:
,docs:
, orchore:
.