Make Codecov upload best effort - #93
Conversation
|
👋 Hello @glenn-jocher, thank you for submitting a
For more guidance, please refer to our Contributing Guide. Don't hesitate to leave a comment if you have any questions. Thank you for contributing to Ultralytics! 🚀 |
UltralyticsAssistant
left a comment
There was a problem hiding this comment.
🔍 PR Review
Made with ❤️ by Ultralytics Actions
No issues found. The workflow change is minimal and aligns with the PR goal of making Codecov uploads best-effort without altering the existing coverage configuration.
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
|
🚀 Merged — thank you @glenn-jocher for improving CI reliability!
This update turns a common CI obstacle into a smoother workflow by keeping Codecov reporting enabled while preventing temporary external upload failures from blocking valid PRs. It helps contributors stay focused on real test and code issues, reduces false negatives, and keeps development moving with greater confidence. Appreciate the thoughtful improvement! 🙌 |
Summary\n- make the Codecov upload step non-blocking\n- keep existing coverage files, OIDC, flags, and plugins settings\n\n## Validation\n- ruby -e 'require "yaml"; YAML.load_file(".github/workflows/ci.yml")'\n- git diff --check
🛠️ PR Summary
Made with ❤️ by Ultralytics Actions
🌟 Summary
Improves CI reliability by making Codecov coverage uploads non-blocking ✅
📊 Key Changes
continue-on-error: trueto the Codecov upload step in GitHub Actions.fail_ci_if_errorfromtruetofalse.🎯 Purpose & Impact