-
Notifications
You must be signed in to change notification settings - Fork 36
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
[FSSDK-10437] jest coverage report job addition #281
[FSSDK-10437] jest coverage report job addition #281
Conversation
Code Coverage Report
Test suite run success258 tests passing in 10 suites. Report generated by 🧪jest coverage report action from 29f939a |
Jest Coverage Report
Test suite run success258 tests passing in 10 suites. Report generated by 🧪jest coverage report action from 6fed881 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. 👍
ArtiomTr/jest-coverage-report-action looks like a good and active project.
Part of me wishes we could collect the --coverage only on the last matrix run (20; currently), then if
1 additional step to unitTests
to publish the report. ...but then I'm just over-engineering for the sake of a 1 additional run.
Summary
Test coverage report job added
Test plan
Existing test should pass
Issues