Skip to content
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.

Sign up for GitHub

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

Jump to bottom

docs: add --per-file#435

Merged
merged 2 commits into from Feb 14, 2023
Merged

docs: add --per-file #435

merged 2 commits into from Feb 14, 2023

Conversation

AriPerkkio
Copy link
Contributor

While rewriting vitest's coverage configuration documentation I noticed the per-file option is not included in c8's documentation. It is implemented though:

c8/lib/commands/check-coverage.js

Lines 46 to 52 in bc347a9

if (argv.perFile) {
map.files().forEach(file => {
checkCoverage(map.fileCoverageFor(file).toSummary(), thresholds, file)
})
} else {
checkCoverage(map.getCoverageSummary(), thresholds)
}

Checklist
  • documentation is changed or added

@bcoe bcoe merged commit 7263c8d into bcoe:main Feb 14, 2023
@bcoe
Copy link
Owner

bcoe commented Feb 14, 2023

@AriPerkkio thank you for the contribution.

@AriPerkkio AriPerkkio deleted the docs/per-file branch February 14, 2023 18:22
mcknasty pushed a commit to mcknasty/c8 that referenced this pull request Feb 2, 2024
@AriPerkkio @bcoe
docs: add --per-file (bcoe#435)
106ee14 
Co-authored-by: Benjamin E. Coe <bencoe@google.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@bcoe bcoe bcoe approved these changes
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet

2 participants
@AriPerkkio @bcoe