Skip to content

Add optional test coverage reports #2517

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

Merged
merged 3 commits into from
Sep 23, 2022
Merged

Add optional test coverage reports #2517

merged 3 commits into from
Sep 23, 2022

Conversation

dcodeIO
Copy link
Member

@dcodeIO dcodeIO commented Sep 22, 2022

Based on #2294, but keeps coverage manual / optional for now.

cc @JesseCodeBones: Wdyt?

  • I've read the contributing guidelines
  • I've added my name and email to the NOTICE file

@JesseCodeBones
Copy link
Contributor

@dcodeIO
I think we need add two things:
1, enable all features during coverage: Link
2, add c8 to the npm dependencies: Link, maybe the package_lock.json also need to be included.
you can chose merge which PR to be merged, Thank you for your advice:)
PR

@dcodeIO
Copy link
Member Author

dcodeIO commented Sep 23, 2022

The idea is to provide a start for those interested in working on coverage. In the typical case of normal development, the additional coverage dependencies are not necessary, hence keeps c8 on-demand. I've documented the remaining bits for now, including to manually enable all (relevant) features. Should get us going, and we can still extend this later :)

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