Skip to content

Overhaul code coverage pipeline and Codecov configuration - #207

Merged
Kyle-Ye merged 5 commits into
mainfrom
codecov-pipeline-overhaul
Feb 25, 2026
Merged

Overhaul code coverage pipeline and Codecov configuration#207
Kyle-Ye merged 5 commits into
mainfrom
codecov-pipeline-overhaul

Conversation

@Kyle-Ye

Copy link
Copy Markdown
Member

Summary

  • Tighten project coverage threshold from 5% to 1%
  • Enable 70% patch coverage target for PR-level feedback on new code
  • Add platform flags (macos, ubuntu, ios) to distinguish coverage uploads
  • Add PR comment configuration with header, diff, flags, files layout
  • Add flag_management with carryforward support for per-flag statuses
  • Upgrade Ubuntu codecov-action from v3 to v5 (align with macOS)
  • Add Codecov upload step to iOS workflow using swift_project parameter

Test plan

  • Verify macOS CI passes and uploads coverage with macos flag
  • Verify Ubuntu CI passes and uploads coverage with ubuntu flag
  • Verify iOS CI passes and uploads coverage with ios flag
  • Verify Codecov PR comment appears with header, diff, flags, files layout
  • Verify patch coverage status check appears on this PR
  • If iOS swift_project auto-discovery fails, switch to explicit xcrun xccov fallback

- Tighten project coverage threshold from 5% to 1%
- Enable 70% patch coverage target for PR feedback
- Add platform flags (macos, ubuntu, ios) to distinguish uploads
- Add PR comment with header, diff, flags, files layout
- Add flag_management with carryforward support
- Upgrade Ubuntu codecov-action from v3 to v5
- Add Codecov upload step to iOS workflow
The swift:6.1.2-jammy Docker container lacks curl and gpg,
which codecov-action v5 requires for downloading and validating
the Codecov CLI binary.
@augmentcode

Copy link
Copy Markdown
🤖 Augment PR Summary

Summary: Updates Codecov configuration and CI workflows to give clearer PR-level coverage feedback across platforms.

  • Enables patch coverage status with a 70% target and tightens the project status drop threshold to 1%.
  • Adds flag_management with carryforward to support per-flag statuses for platform-specific uploads.
  • Configures Codecov PR comments to include header, diff, flags, and files sections.
  • Upgrades Ubuntu’s Codecov action to v5 and adds platform flags for macOS/Ubuntu/iOS uploads.
  • Adds a Codecov upload step to the iOS workflow (including swift_project metadata).

🤖 Was this summary useful? React with 👍 or 👎

@augmentcodeaugmentcodeBot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Review completed. 1 suggestions posted.

Fix All in Augment

Comment augment review to trigger a new review at any time.

Comment thread.github/workflows/ubuntu.yml
@codecov

codecovBot commented Feb 25, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 29.01%. Comparing base (1a57428) to head (1a91f6a).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@ Coverage Diff @@## main #207 +/- ##
===========================================
+ Coverage 10.12% 29.01% +18.89% 
===========================================
Files 28 70 +42 Lines 820 2550 +1730 ===========================================
+ Hits 83 740 +657 - Misses 737 1810 +1073 
FlagCoverage Δ
ios29.03% <ø> (?)
macos10.14% <ø> (?)
ubuntu0.00% <ø> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.
see 43 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

swift_project doesn't auto-discover xcodebuild coverage from derived
data. Extract coverage explicitly from the xcresult bundle using
xcrun xccov instead.
xcrun xccov JSON format isn't recognized by Codecov's parser.
Switch to xcrun llvm-cov show to generate coverage.txt in the same
format used by macOS and Ubuntu workflows.
The statuses in flag_management.default_rules duplicated the ones in
coverage.status, causing redundant codecov/project/ and codecov/patch/
checks (with trailing slash). Keep only carryforward in flag_management
and let coverage.status handle all status checks.
@Kyle-Ye
Kyle-Ye merged commit b448d85 into mainFeb 25, 2026
6 checks passed
@Kyle-Ye
Kyle-Ye deleted the codecov-pipeline-overhaul branch February 25, 2026 07:00
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant

@Kyle-Ye