Skip to content

Conversation

@mdepasca-lrz
Copy link

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:

  • The pull request only contains commits that are focused and relevant to this change.
  • I have added appropriate tests that cover the new/changed functionality.
  • I have updated the documentation to reflect these changes.
  • I have added entries to the changelog for any noteworthy additions, changes, fixes, or
    removals.
  • I have added migration instructions to the upgrade guide (if needed).
  • The changes follow the project's style guidelines and introduce no new warnings.
  • The changes are fully tested and pass the CI checks.
  • I have reviewed my own code changes.

@ystade
Copy link
Collaborator

ystade commented Oct 29, 2025

As stated here the codecov action is currently not expected to work on forks, unfortunately.

@kayaercument
Copy link
Collaborator

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 🤞 .

@ystade
Copy link
Collaborator

ystade commented Oct 29, 2025

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.

@thomasrockhu-codecov
Copy link

@kayaercument @mdepasca-lrz do you mind re-running CI again? I think I made the fix for you

@burgholzer
Copy link
Contributor

@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.
Also retriggered in #154, which also still shows the same error.

@kayaercument
Copy link
Collaborator

Hey @thomasrockhu-codecov 👋 ,

Could you please take a look at it again? It still does not work

@thomasrockhu-codecov
Copy link

@kayaercument sorry about the delay here, it should work if you re-run CI now

@kayaercument
Copy link
Collaborator

@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?

@burgholzer
Copy link
Contributor

Yeah, #154 now also has coverage data.
Thanks @thomasrockhu-codecov 🙏🏼

Closing this PR now.

@burgholzer burgholzer closed this Nov 12, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants