Here you go: 30758f30538bc21f7d082e2c76ffa496c1990992
Thank you for your patience.
Sorry, I don’t think
enabled is valid. Since you have settings, it’s on. Try removing that key?
What do you mean? Move
# configuration related to pull request comments comment: no # do not comment PR with the result coverage: range: 50..90 # coverage lower than 50 is red, higher than 90 green, between color code status: project: # settings affecting project coverage default: target: auto # auto % coverage target threshold: 5% # allow for 5% reduction of coverage without failing # do not run coverage on patch nor changes patch: false
I am going to try this last option now. Removing the
enabled: yes did not work either, in case it helps, here is the sha c5ec4a094182414cc99231f696d3d3a8b372a68f.
In around 10-15 minutes I will reply again, hopefully saying it works, otherwise with a new sha.
Thanks for your patience.
Unfortunately it did not work. Here is the sha be4102955978d03da4e7d9451d16154acaa65f6b
Could it be due to some other misconfiguration outside the yaml file?
I met the same issue. yaml file is valid, but
codecov/patch is still there.
It works! Thanks a lot!
codecov.yml instead of
.codecov.yml solved the problem, thanks a lot.
It would be great if you could update the docs or explain how would we update them if it were possible. There are various references in the docs to
.codecov.yml and there is a question about this in the FAQs where it is clearly said the initial dot in the name should not affect.
There should be a suggest edits link at the top right of every docs page, i will go though and adjust any references i find on Monday.
Glad that fixed it for you.
It looks like our old code has a bug where it will find a file that starts with a dot, but the new code fixes this. Since the new code will be fully deployed soon, I’m going to hold off on docs edits for now. I’ll keep an eye on the issue and see how many folks hit it.
Edit: I’ve updated all docs pages to clarify that the file needs to be named
codecov.yml (no period)
I also have problems turning off the patch status in https://github.com/raiden-network/raiden/commit/98d4fa02439a5e255d4c8092f8c9d789f9394ae4. What’s the current state of this? Is the documentation up to date? Does
.codecov.yml work reliably?
The file needs to be named
codecov.yml (no period). I believe I’ve updated the docs everywhere for this, please let me know if I missed one.
My commit above was written backwards, let me correct.
Sorry, I misunderstood your ask. Your YAML is being ignored because your flags section is invalid. It should be:
flags: - unit - integration
Thanks for the feedback @drazisil!
The yml properly validates now and has been renamed to
codecov.yml. Unfortunately, I still get the “codecov/patch” status:
I tried to turn it off as described in https://docs.codecov.io/docs/commit-status#section-disabling-a-status.
It’s still failing validation internally, I left a comment on the PR.
As I suspected, this is a bug in the legacy version of our notify task, that is fixed in in our new version. Since everyone will be on the new version in the next week or two, we have opted not to fix the old version. Once all your jobs are running though the new notify task (it’s a percentage currently) the patch status will go away.