Make requirement that coverage *must* be in XML more obvious #456
Assignees
Comments
This comment has been minimized.
This comment has been minimized.
|
I was specifying |
This comment has been minimized.
This comment has been minimized.
|
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
But this doesn't seem to actually be the case. Could the documentation be updated to clarify what actually needs to happen? |
daniellockyer
added a commit
to TryGhost/framework
that referenced
this issue
Oct 20, 2021
refs codecov/codecov-action#456 - turns out the coverage files need to be in XML for them to work with codecov, so this commit adds cobertura (XML) as a reporter
daniellockyer
added a commit
to TryGhost/framework
that referenced
this issue
Oct 20, 2021
refs codecov/codecov-action#456 - turns out the coverage files need to be in XML for them to work with codecov, so this commit adds cobertura (XML) as a reporter
|
Ah, good to know... |
|
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.
|
p-j-smith
pushed a commit
to p-j-smith/lipyphilic
that referenced
this issue
Dec 29, 2021
p-j-smith
pushed a commit
to p-j-smith/lipyphilic
that referenced
this issue
Dec 29, 2021
|
The uploader now runs |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Spent a good hour on this until I realized that maybe everyone was running
coverage xmlfor a reason. It is only implied and never out-right mentioned in the README.The text was updated successfully, but these errors were encountered: