-
Notifications
You must be signed in to change notification settings - Fork 170
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
Upgrade kover #1782
Upgrade kover #1782
Conversation
ea5fc04
to
9ff3120
Compare
33fab91
to
80c69e7
Compare
📱 Scan the QR code below to install the build (arm64 only) for this PR. |
OK, I have now the same error than when running locally:
Maybe we could just remove the |
I think it's not taking into account any screenshot test, it's the same issue we had before. All views have 0% of coverage. |
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, let's wait for the new test run to finish and check that codecov is fine before merging.
It wasn't taking into account any test result from the debug runs or screenshot tests Co-authored-by: Jorge Martin Espinosa <jorgem@element.io>
d10ed56
to
45fd075
Compare
Quality Gate passedKudos, no new issues were introduced! 0 New issues |
Migrate to Kover 0.7.5
Report are now located in the
:app
module.Verify and Html report tested OK locally.
First commits are just removing old empty modules.