-
Notifications
You must be signed in to change notification settings - Fork 2
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
Upgrade to new CodeCov action #65
Labels
enhancement
New feature or request
Comments
Thank you @Supereg for documenting this. My gut assumption is that running In the end it might just be a thing that needs to be fixed as part of your bug report in codecov/codecov-action#1367 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Problem
We are currently using the
v3
version of the codecov action. A newer releasev4
removed a feature that we relied on, namely converting.xcresult
coverage format to one that codecov understands. This was removed without a proper replacement. We have reported our situation here: codecov/codecov-action#1367. Our motivation to upgrade to v4 is mainly driven by the fact that tokenless uploads were randomly broken in v3.Their new approach uses
llvm-cov
which uses.profdata
files sourced from the derived data folder (the path is hardcoded to~/Library/Developer/Xcode/DerivedData
), see https://github.com/codecov/uploader/pull/968/files. They scan for all.app
,.framework
and.test
folders in the derived data subfolders.Solution
There are two ways forward here:
We could use a tool like xcresultparser (available via brew) to do the conversion ourself into the Cobertura XML format. Drawback would be that this introduces yet another tool, not a 100% conversion (authors note Note that some data in this file is currently fake) and relying on the xccov format which might randomly break between Xcode releases (seemingly also the reason why there is not a large adoption for that?).
Another approach could be to execute the
xcrun llvm-cov ...
commands ourself?If we wanted to adapt our infrastructure for the new codecov infrastructure. We would need to make sure that the necessary
profdata
files are uploaded as artifacts (such that the collect coverage reports action can use it). We are probably fine uploading a.xctest
archive (not sure how it differs from.xcresult
).Additional context
Related PRs around cutest coverage conversion:
Code of Conduct
The text was updated successfully, but these errors were encountered: