-
Notifications
You must be signed in to change notification settings - Fork 779
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
Coverage reports on front-end unit tests #2335
Conversation
Codecov Report
@@ Coverage Diff @@
## master #2335 +/- ##
=========================================
Coverage ? 58.83%
=========================================
Files ? 77
Lines ? 4715
Branches ? 0
=========================================
Hits ? 2774
Misses ? 1941
Partials ? 0 Continue to review full report at Codecov.
|
@galipremsagar @RishabhJain2018 Please review |
@Sanji515 , It says we have no coverage report?? I think we need to fix that aswell? |
It's saying because it don't have any previous data to match with in master branch, so when it will be merged then it will show diff coverage report for all other PR's |
Cool 👍 |
@Sanji515, can you update your branch so that we can merge it. This looks good to me. |
looks good. Please merge this @RishabhJain2018 |
No description provided.