Skip to content
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 Codecov misconfigured #3463

Open
timkimadobe opened this issue Feb 26, 2025 · 0 comments
Open

CI Codecov misconfigured #3463

timkimadobe opened this issue Feb 26, 2025 · 0 comments

Comments

@timkimadobe
Copy link

Describe the bug
Hi Home Assistant team,

Sorry I couldn't find an issue type that fit CI setup so I put it under iOS.

I noticed that in your active CI script, Codecov action is configured with non-existent params:

xcode: true

That is, xcode and xcode_archive_path are not valid inputs to the workflow (at least as of the version in use v5.3.1).

Looking at a recent run, this is shown in the logs:

Warning: Unexpected input(s) 'xcode', 'xcode_archive_path', valid inputs are ['base_sha', 'binary', 'codecov_yml_path', 'commit_parent', 'directory', 'disable_file_fixes', 'disable_search', 'disable_safe_directory', 'disable_telem', 'dry_run', 'env_vars', 'exclude', 'fail_ci_if_error', 'files', 'flags', 'force', 'git_service', 'gcov_args', 'gcov_executable', 'gcov_ignore', 'gcov_include', 'handle_no_reports_found', 'job_code', 'name', 'network_filter', 'network_prefix', 'os', 'override_branch', 'override_build', 'override_build_url', 'override_commit', 'override_pr', 'plugins', 'report_code', 'report_type', 'root_dir', 'run_command', 'skip_validation', 'slug', 'swift_project', 'token', 'url', 'use_legacy_upload_endpoint', 'use_oidc', 'use_pypi', 'verbose', 'version', 'working-directory']

...
// It looks like coverage data is being generated elsewhere - I think `vendor/bundle`? but Codecov is currently looking at the default path
error: Failed to load coverage: '/Users/runner/Library/Developer/Xcode/DerivedData/HomeAssistant-boibdhgrtssbkjefnppeafyunwcq/Build/Products/Debug-iphonesimulator/HomeAssistant-Tests-Shared.xctest/Frameworks/ObjcExceptionBridging.framework/ObjcExceptionBridging': No coverage data found
warning - 2025-02-26 06:19:42,890 -- llvm-cov failed to produce results for /Users/runner/Library/Developer/Xcode/DerivedData/HomeAssistant-boibdhgrtssbkjefnppeafyunwcq/Build/Products/Debug-iphonesimulator/HomeAssistant-Tests-Shared.xctest/Frameworks/ObjcExceptionBridging.framework/ObjcExceptionBridging

...
// With v5 Codecov I believe you are now able to specify at your org level if you want tokenless or token upload
 error - 2025-02-26 06:19:45,207 -- Upload failed: {"message":"Token required because branch is protected"}

This means that code coverage isn't actually being reported as expected, however, I'm not sure if your team is still using Codecov for coverage - just wanted to bring this to your attention!

To Reproduce

Expected behavior

Screenshots

Additional context

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant