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

Add file based coverage report #8

Merged
merged 37 commits into from
Jul 8, 2022
Merged

Add file based coverage report #8

merged 37 commits into from
Jul 8, 2022

Conversation

davelosert
Copy link
Owner

No description provided.

@github-actions
Copy link
Contributor

github-actions bot commented Jun 20, 2022 •
edited
Loading

Coverage Summary

Status Category Percentage Covered / Total
🔵 Lines 61.86% 391 / 632
🔵 Statements 61.86% 391 / 632
🔵 Functions 56.25% 9 / 16
🔵 Branches 89.09% 49 / 55
File Coverage
File Stmts % Branch % Funcs % Lines Uncovered Lines
src/generateFileCoverageHtml.ts 0% 0% 0% 0% 1-70
src/generateFileUrl.ts 0% 0% 0% 0% 1-18
src/generateSummaryTableHtml.test.ts 100% 100% 100% 100%
src/generateSummaryTableHtml.ts 98.48% 100% 66.66% 98.48% 59
src/getUncoveredLinesFromStatements.test.ts 100% 100% 100% 100%
src/getUncoveredLinesFromStatements.ts 100% 100% 100% 100%
src/icons.ts 100% 100% 100% 100%
src/index.ts 0% 0% 0% 0% 1-34
src/parseJsonReports.ts 0% 0% 0% 0% 1-50
src/parseThresholds.test.ts 100% 100% 100% 100%
src/parseThresholds.ts 100% 100% 100% 100%
src/writeSummaryToPR.ts 0% 0% 0% 0% 1-60
src/testFolder/moreNesting/untestedFile.ts 0% 0% 0% 0% 1-8

@davelosert
feat: Adds first implementation of uncovered lines
b3c497b
@davelosert
chore: Renamings for better readbility
02b087e
@davelosert
refactor: Generalizes summary-function to be reused for file-based re…
9b754f4 
…ports
@davelosert
fix: Changes core to correct warning
e9335ba
@davelosert
feat: Adds detailed file report
fec1128
@davelosert
feat: switches to use own html table template
51ac77c
@davelosert
fix: Fixes wrong quote style
2271008
@davelosert
debug: logs fileTable output
a371631
@davelosert
fix: Removes safeHtml and unecessary comma
b427679
@davelosert
chore: Fixes tr to td
57e413e
@davelosert
fix: Adjusts url for file
fdc2b7a
@davelosert
fix: Removes unecessary commatas in report
6632636
@davelosert
chore: Improves typing of fileCoverageHtml
6391569
@davelosert
fix: Switches summary table to html template to have better alignments
33b3268
@davelosert
fix: Inserts missing trs for summary report
4932bfe
@davelosert
fix: Removes unecessary column
4120213
@davelosert
ci: Try actions checkout v3 to have correct sha
edb7715
@davelosert
debug: Log SHAs
ea1b55f
@davelosert
chore: Adds error-handling for json file parsing
d968fba
@davelosert
debug: Removes console.log statements
743e731
@davelosert
test: see if action produces correct warning
5b90fb7
@davelosert
fix: Adds missing await statement to error handling
dc01282
@davelosert
chore: Included stack trace to errors
a0098aa
@davelosert
fix: Makes file coverage report use keys from summary
fd45f7c
@davelosert
fix: Makes filecoverage work without linecoverage
c9c7193
@davelosert
chore: Removes duplicated error message
3f064d7
@davelosert
fix: Filter correct total key
02409ff
@davelosert
debug: Test error on wrong summary path
7a32d70
@davelosert davelosert marked this pull request as ready for review July 8, 2022 14:11
@davelosert davelosert changed the title draft: some draft to add missing lines in the report Add file based coverage report Jul 8, 2022
@davelosert davelosert merged commit 34f51ec into main Jul 8, 2022
@github-actions
Copy link
Contributor

github-actions bot commented Jul 8, 2022

🎉 This PR is included in version 1.2.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

@davelosert davelosert deleted the add-missing-lines-report branch August 7, 2023 06:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
released
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@davelosert