Skip to content

Project coverage may move within a percent before its status fails - #2172

Merged
Rafael-SOWNet merged 1 commit into
masterfrom
project-coverage-may-move-within-a-percent
Oct 27, 2026
Merged

Rafael-SOWNet merged 1 commit into
masterfrom
project-coverage-may-move-within-a-percent

Conversation

@Rafael-SOWNet

Copy link
Copy Markdown
Member

With no threshold, codecov/project fails on any fall at all from the base commit's figure. Now that reports reach Codecov (#2162, #2165), that turns ordinary changes red: #2171 was red at −0.02%, five lines that a new guard returns before.

A threshold of 1% lets that movement through. It still fails what the status is for, such as a test leg whose report goes empty, which moved the figure from 86.41% to 0.38% before #2165.

Codecov's validator accepts the file and reads the threshold back as 1.0. patch and changes stay off, as before.

🤖 Generated with Claude Code

https://claude.ai/code/session_012sonx8iAspMiwRwokT1Ura

With no threshold, codecov/project fails on any fall at all from the base
commit's figure. Now that reports reach Codecov (#2162, #2165), that turns
ordinary changes red: #2171 was at -0.02%, five lines that a new guard
returns before. A threshold of 1% lets that movement through and still
fails what the status is for, such as a test leg whose report goes empty,
which moved the figure from 86.41% to 0.38%. Codecov's validator accepts
the file and reads the threshold back as 1.0.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_012sonx8iAspMiwRwokT1Ura
@Rafael-SOWNet Rafael-SOWNet added this to the 2.7.0 milestone Oct 27, 2026
@Rafael-SOWNet Rafael-SOWNet moved this from Todo to In review in AngouriMath Oct 27, 2026
@codecov

codecov Bot commented Oct 27, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 86.41%. Comparing base (2d7d31e) to head (9b8fedb).
⚠️ Report is 1 commits behind head on master.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #2172      +/-   ##
==========================================
- Coverage   86.42%   86.41%   -0.01%     
==========================================
  Files         295      295              
  Lines       64808    64808              
  Branches    19060    19060              
==========================================
- Hits        56011    56007       -4     
- Misses       4767     4771       +4     
  Partials     4030     4030              
Flag Coverage Δ
cpp 37.77% <ø> (ø)
csharp 86.99% <ø> (-0.01%) ⬇️
fsharp 32.97% <ø> (ø)
interactive 40.72% <ø> (ø)

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

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

@Rafael-SOWNet
Rafael-SOWNet merged commit 8a85f8c into master Oct 27, 2026
28 checks passed
@Rafael-SOWNet Rafael-SOWNet moved this from In review to Done in AngouriMath Oct 27, 2026
@github-project-automation github-project-automation Bot moved this from In review to Done in AngouriMath Oct 27, 2026
@Rafael-SOWNet
Rafael-SOWNet deleted the project-coverage-may-move-within-a-percent branch October 28, 2026 10:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

1 participant