Adds the simple codecov format to make using the codecov action easier #616
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
As the codecov API has changed, direct uploads to codecov are currently somewhat flaky and might stop working in the future altogether when the old API endpoints are deactivated. A current workaround is using
to_coberturato generate a file and then upload it using the codecov action, which however introduces thexml2dependency (e.g. used in #584 and #568).This PR proposes
to_simple_codecovto solve this in a simpler way. The function returns the very simple Codecov custom format (also seems similar (but not the same) to the existing codecov format). By writing this to a file we can upload the report via the GH action (on other services via the CLI) and do not have to worry about the API version anymore.A codecov report based on a report file from
to_simple_codecovcan be found here.This is very much a first proposal only and the question, whether you find this useful, so no documentation and tests yet.