-
Notifications
You must be signed in to change notification settings - Fork 1.5k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
CI: Attempt to enable codecov again #7034
Conversation
If we want to fix the coverage it is likely you need to install the codecov app for the repository. But I mentioned this several times in the past already - see https://trac.cppcheck.net/ticket/12112. |
I have installed the app now |
I don't know what to do.. do you have an idea how to fix it? Is this something you could get with higher permissions or whatever? |
ok I have added the CODECOV_TOKEN I think.. |
Updating to v4 without any other changes will fix the PRs - but it will fail after the merge: codecov/codecov-action#1359 (comment). |
To fix main see codecov/codecov-action#1359 (comment). See also further comments in that ticket. |
It seems to work now? |
or do you think it might fail on main still? |
Based on the information which is available it should not. Please remember to restore the deleted files before merging. |
Yay, it worked! Thanks. Let's hope the sporadic upload failures are also fixed now since we have enabled the app. Let's give it a few days and then we can close that ticket. |
No description provided.