feat(learn): Collecting code coverage #7006
Open
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
|
This is currently marked as draft until the next semver-minor release of Node.js |
apps/site/pages/en/learn/test-runner/collecting-code-coverage.md
Outdated
Show resolved
Hide resolved
apps/site/pages/en/learn/test-runner/collecting-code-coverage.md
Outdated
Show resolved
Hide resolved
apps/site/pages/en/learn/test-runner/collecting-code-coverage.md
Outdated
Show resolved
Hide resolved
apps/site/pages/en/learn/test-runner/collecting-code-coverage.md
Outdated
Show resolved
Hide resolved
apps/site/pages/en/learn/test-runner/collecting-code-coverage.md
Outdated
Show resolved
Hide resolved
apps/site/pages/en/learn/test-runner/collecting-code-coverage.md
Outdated
Show resolved
Hide resolved
apps/site/pages/en/learn/test-runner/collecting-code-coverage.md
Outdated
Show resolved
Hide resolved
RedYetiDev
force-pushed
the
code-coverage-learn
branch
from
ee241d0
to
f26b449
Compare
|
Deployment failed with the following error: |
AugustinMauroy
approved these changes
Aug 24, 2024
apps/site/pages/en/learn/test-runner/collecting-code-coverage.md
Outdated
Show resolved
Hide resolved
mikeesto
approved these changes
Aug 25, 2024
apps/site/pages/en/learn/test-runner/collecting-code-coverage.md
Outdated
Show resolved
Hide resolved
Co-authored-by: Michael Esteban <mickel13@gmail.com> Signed-off-by: Aviv Keller <38299977+RedYetiDev@users.noreply.github.com>
|
I've undrafted this, as the release proposal for v22.8.0 is ready at nodejs/node#54560, but it still shouldn't land until that does. The expected release date is |
aymen94
approved these changes
Aug 26, 2024
github-actions
bot
removed
the
github_actions:pull-request
Unit Test Coverage Report
Unit Test Report
|
bmuenzenmeyer
approved these changes
Aug 26, 2024
|
Can we have last review form @nodejs/test_runner |
mcollina
reviewed
Sep 2, 2024
mcollina
requested changes
Sep 2, 2024
|
I think this is okay to land once v22.8.0 releases, as it specifies that the |
|
I think it's better to hold off until it's stable. |
AugustinMauroy
added
content
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
This PR introduces a learning article on collecting code coverage in Node.js. However, it should not be merged until support for thresholds is included in the next semver-minor release, which is expected to be next week.
Related Issues
Support for code coverage: nodejs/node#46017
Support for code coverage inclusion/exclusion: nodejs/node#53553
Support for code coverage thresholds: nodejs/node#54429
Check List
npm run formatto ensure the code follows the style guide.npm run testto check if all tests are passing.npx turbo buildto check if the website builds without errors.