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.

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

[Issue #1842]: Add jest coverage report in PR #1843

Merged
merged 3 commits into from
Apr 29, 2024

Conversation

rylew1
Copy link
Collaborator

@rylew1 rylew1 commented Apr 25, 2024

Summary

Fixes #1842

Time to review: 2 min

Changes proposed

I believe subsequent PRs should show the diffs in coverage report that the respective PR introduces - for example this is a snapshot of a PR from the template project
image

  • this is a much smaller project so it's easier over there to get above 90% coverage

Copy link

Coverage report for ./frontend

St.
Category Percentage Covered / Total
🟢 Statements 83.83% 850/1014
🟡 Branches 64.58% 217/336
🟡 Functions 75.34% 165/219
🟢 Lines 83.83% 793/946

Test suite run success

165 tests passing in 55 suites.

Report generated by 🧪jest coverage report action from 978f187

@rylew1 rylew1 merged commit 14ce419 into main Apr 29, 2024
16 checks passed
@rylew1 rylew1 deleted the rylew/1842-jest-coverage-report-in-pr branch April 29, 2024 21:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Task]: Setup jest coverage report github action to display coverage report in PR
2 participants