codecov
Make requirement that coverage *must* be in XML more obvious
Spent a good hour on this until I realized that maybe everyone was running coverage xml for a reason. It is only implied and never out-right mentioned in the README.
Found that the coverage file doesn't need to be an XML file.
Just the .coverge file is not considered as a 'coverage file' like the old bash uploader.
You must specify it manually in config files: .coverage, no need to run coverage xml.
I was specifying files: .coverage and it was being uploaded, but it was considered corrupt (I don't remember the exact error message at this point).
It works for me, but now it looks like there are more bugs than I thought...
Oh, I misunderstood you, yes this file isn't processed correctly, you are right.
------------------ Original ------------------ From: @.>; Date: 2021年8月14日(星期六) 晚上9:56 To: @.>; Cc: @.>; @.>; Subject: Re: [codecov/codecov-action] Make requirement that coverage must be in XML more obvious (#456)
I was specifying files: .coverage and it was being uploaded, but it was considered corrupt (I don't remember the exact error message at this point).
— You are receiving this because you commented. Reply to this email directly, view it on GitHub, or unsubscribe.
The Python example suggests that codecov will run something called coveragexml for us which does this job:
Codecov will call coveragexml -i automatically to generate the coverage xml output, which will be archived and processed server side.
But this doesn't seem to actually be the case. Could the documentation be updated to clarify what actually needs to happen?
It works fine with JSON for me:
[2021-11-02T07:58:05.090Z] ['info'] => Found 1 possible coverage files:
./coverage/coverage.json
[2021-11-02T07:58:05.090Z] ['verbose'] End of network processing
[2021-11-02T07:58:05.090Z] ['info'] Processing ./coverage/coverage.json...
[2021-11-02T07:58:05.094Z] ['info'] Using manual override from args.
The uploader now runs coverage xml by default if coverage is installed (a dep of pytest-cov)
