Skip to content

Allow 0.1% codecov project coverage threshold - #152

Merged
jas88 merged 1 commit into
mainfrom
codecov-threshold
Oct 14, 2026
Merged

jas88 merged 1 commit into
mainfrom
codecov-threshold

Conversation

@jas88

@jas88 jas88 commented Oct 14, 2026 •

Copy link
Copy Markdown
Owner

Summary

codecov/project uses a zero threshold, so any drop fails it. Coverage fluctuates between runs even when no source changes: across the last 12 dependency-only commits on main it moved between -0.05% and +0.03%, and the check failed on 6 of them. #150 hit the same thing (-0.01%, from StorageCommitmentRequest.cs and Dwt97.cs, neither touched by the PR).

Adds .github/codecov.yml allowing a 0.1% project drop, which still catches real regressions. The patch check is unchanged.

The underlying run-to-run variation (hits dropping in deterministic tests such as StorageCommitmentTests) is not addressed here.


Summary by cubic

Adds a 0.1% coverage threshold for codecov's project check so small run-to-run variations no longer fail it.

Coverage fluctuates up to ±0.05% between identical runs, failing the check on about half of recent main commits even with no source changes. The new threshold still catches real regressions, and the patch check is unchanged.

Written for commit 1aec711. Summary will update on new commits.

View guided diff Turn on auto-fix

Project coverage fluctuates by up to 0.05% between runs with no source
changes, failing codecov/project on about half of recent main commits.

@cubic-dev-ai cubic-dev-ai Bot 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.

No issues found across 1 file

View guided diff | Turn on auto-fix | Re-trigger cubic

@codecov

codecov Bot commented Oct 14, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 65.64%. Comparing base (7886aed) to head (1aec711).

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #152      +/-   ##
==========================================
+ Coverage   65.61%   65.64%   +0.02%     
==========================================
  Files         337      337              
  Lines       30420    30420              
  Branches     5710     5710              
==========================================
+ Hits        19961    19969       +8     
+ Misses       9038     9032       -6     
+ Partials     1421     1419       -2     
Flag Coverage Δ
unittests 65.64% <ø> (+0.02%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

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

@jas88
jas88 merged commit e765c72 into main Oct 14, 2026
6 checks passed
@jas88
jas88 deleted the codecov-threshold branch October 14, 2026 01:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant