Skip to content

Add .codecov.yml configuration#109

Closed
eedri wants to merge 1 commit into
konflux-ci:mainfrom
eedri:add-codecov-yml
Closed

Add .codecov.yml configuration#109
eedri wants to merge 1 commit into
konflux-ci:mainfrom
eedri:add-codecov-yml

Conversation

@eedri

@eedri eedri commented Jun 11, 2026

Copy link
Copy Markdown

Summary

  • Adds .codecov.yml with standard Konflux coverage settings
  • Project coverage target: auto (threshold: 1%)
  • Patch coverage target: 80%
  • All checks are informational — coverage results are reported but never block CI

Details

Setting Value
require_ci_to_pass false
Project target auto (tracks current coverage)
Project threshold 1% (allowed drop)
Patch target 80% (new lines)
informational true (never blocks PRs)
require_changes true (only comments when coverage changes)

Reference

Part of org-wide codecov rollout (KFLUXDP-1020)

🤖 Generated with Claude Code

Adds Codecov coverage configuration with:
- Project target: auto (threshold: 1%)
- Patch target: 80%
- All checks are informational (won't block CI)

Signed-off-by: Eyal Edri <eedri@redhat.com>
@codecov-commenter

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 0.00%. Comparing base (56f27e8) to head (9ed87f6).
⚠️ Report is 14 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@          Coverage Diff          @@
##            main    #109   +/-   ##
=====================================
  Coverage   0.00%   0.00%           
=====================================
  Files          6       6           
  Lines        470     505   +35     
=====================================
- Misses       470     505   +35     
Flag Coverage Δ
unit-tests 0.00% <ø> (ø)

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


Continue to review full report in Codecov by Harness.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update dd8c9b1...9ed87f6. Read the comment docs.

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

@qodo-app-for-konflux-ci

qodo-app-for-konflux-ci Bot commented Jun 11, 2026

Copy link
Copy Markdown

Code Review by Qodo

🐞 Bugs (0) 📘 Rule violations (0) 📎 Requirement gaps (0)

Grey Divider

Great, no issues found!

Qodo reviewed your code and found no material issues that require review

Grey Divider

Qodo Logo

@qodo-app-for-konflux-ci

Copy link
Copy Markdown

PR Summary by Qodo

Add Codecov configuration (.codecov.yml) with informational coverage checks
⚙️ Configuration changes 🕐 Less than 10 minutes

Grey Divider

Walkthroughs

Description
• Add Codecov YAML configuration for project and patch coverage targets.
• Make all coverage statuses informational to avoid blocking CI/PRs.
• Configure Codecov comments to post only when coverage changes.
Diagram
graph TD
  A[CI run] --> B[(Coverage report)] --> C{{Codecov}} --> D[Statuses: informational]
  C --> E[PR comment]
  F[".codecov.yml"] -. configures .-> C

  subgraph Legend
    direction LR
    _proc[Process] ~~~ _art[(Artifact)] ~~~ _ext{{External}} ~~~ _cfg["Config file"]
  end
Loading
High-Level Assessment

The following are alternative approaches to this PR:

1. Enforce coverage as required checks
  • ➕ Prevents coverage regressions from merging
  • ➕ Encourages incremental test improvements
  • ➖ Can create friction/noise for small PRs and legacy codebases
  • ➖ May require tuning exclusions and baselines before enabling
2. Start with informational, then phase into enforcement
  • ➕ Low-risk rollout that surfaces current coverage trends first
  • ➕ Allows teams to calibrate targets and exclusions over time
  • ➖ Regressions can still merge during the informational period
  • ➖ Requires follow-up work to flip checks to blocking

Recommendation: The chosen informational-only configuration is a good default for an org-wide rollout: it provides visibility without blocking delivery. Consider adding a follow-up milestone to optionally transition patch coverage to blocking once baseline noise/exclusions are validated.

Grey Divider

File Changes

Other (1)
.codecov.yml Add Codecov thresholds and comment behavior (informational checks) +18/-0

Add Codecov thresholds and comment behavior (informational checks)

• Introduces Codecov configuration with non-blocking project and patch coverage statuses. Sets project coverage to track current coverage (auto) with a 1% allowed drop, and patch coverage target to 80%. Configures comments to use a standard layout and only post when coverage changes.

.codecov.yml


Grey Divider

Qodo Logo

@eedri

eedri commented Jun 18, 2026

Copy link
Copy Markdown
Author

Closing — repo has no application code, skipping codecov config.

@eedri eedri closed this Jun 18, 2026
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.

2 participants