-
Notifications
You must be signed in to change notification settings - Fork 670
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
[Discussion] Multiple code coverage report comments #488
Comments
eXamadeus
changed the title
Multiple code coverage report comments
[Discussion] Multiple code coverage report comments
Feb 9, 2021
Sounds good to me! |
I killed these sometime back. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Multiple code coverage reports are used in PRs
Reselect should probably only have one. Right now it's using both
Coveralls
andCodecov.io
. It appears thatCoveralls
was not running for some time and is the source of most of the duplicates.Unless there are dissenting opinions, I suggest we remove the
Coveralls
action from the workflow and leave the only coverage reporter to beCodecov.io
.Spawned from comment: @markerikson in #486 (comment)
The text was updated successfully, but these errors were encountered: