🧪 Testing code cov - not meant to be merge#232
Conversation
|
As stated here the codecov action is currently not expected to work on forks, unfortunately. |
I checked the issues in the codecov-action repository, and the very same error was reported previously. They mentioned a sync problem. Just to make sure, I created another fork and PR. I contacted the Codecov team about this. Hopefully, it will be resolved soon 🤞 . |
Awesome, many thanks. Please, keep us posted. |
|
@kayaercument @mdepasca-lrz do you mind re-running CI again? I think I made the fix for you |
Retriggered the old job, even pushed a new commit here; still fails with the same error. |
|
Hey @thomasrockhu-codecov 👋 , Could you please take a look at it again? It still does not work |
|
@kayaercument sorry about the delay here, it should work if you re-run CI now |
Thank you very much @thomasrockhu-codecov ! The test is running without a problem. @ystade @burgholzer Can you confirm? |
|
Yeah, #154 now also has coverage data. Closing this PR now. |
Fork-based PRs cannot access the CODECOV_TOKEN secret, so coverage information would normally be submitted via the tokenless upload flow. This somehow stopped working, even though tokenless uploads are enabled on the codecov side.A Related issues: - codecov/codecov-action#1782 - codecov/codecov-action#1842 - Munich-Quantum-Software-Stack/QDMI#232 Apparently the solution there was to ping someone from the codecov team to "sync" the repo config. As a temporary measure, this PR disables the `fail_ci_if_error` option so we can still merge external PRs while the underlaying issue is solved.
Description
Please include a summary of the change and which issue is fixed. Please also include relevant
motivation and context. List any dependencies that are required for this change.
Fixes #(issue)
Checklist:
removals.