Skip to content

chore: add codecov coverage tracking - #496

Merged
svor merged 1 commit into
mainfrom
add-codecov-coverage
Oct 27, 2026
Merged

svor merged 1 commit into
mainfrom
add-codecov-coverage

Conversation

@mkuznyetsov

Copy link
Copy Markdown
Contributor

Summary

This PR adds Codecov integration for automated code coverage tracking and reporting.

Changes

  • GitHub Actions: Updated .github/workflows/pr-checks.yml

    • Added push trigger for main branch (establishes baseline coverage for PR comparisons)
    • Added id-token: write permission for OIDC authentication
    • Added Codecov upload step using OIDC (no secrets required)
    • Tagged uploads with unit-tests flag
  • Codecov Configuration: Added .codecov.yml

    • Ignore patterns for test files, compiled output, and dependencies
    • Carryforward flags to prevent coverage drops when CI fails
    • Auto coverage targets with 1% threshold
    • PR comment layout configuration

Benefits

✅ Automated coverage tracking on every PR and main branch push
✅ PR comments showing coverage changes
✅ Coverage trend tracking over time
✅ No secrets needed (OIDC authentication)

Prerequisites

Before merging, ensure:

  1. Repository is added to https://app.codecov.io under the devfile organization
  2. GitHub OIDC is enabled in Codecov (enabled by default for new repos)

Testing

  • ✅ Local test run: 13 suites, 97 tests passed, 100% coverage
  • ✅ Coverage file generated at expected path: coverage/lcov.info
  • ✅ Workflow configuration validated

🤖 Generated with Claude Code

@coderabbitai

coderabbitai Bot commented Oct 26, 2026 •

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

Run configuration
  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: fd92d735-e0ef-4218-8892-35d2db82c243
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@codecov

codecov Bot commented Oct 26, 2026

Copy link
Copy Markdown

Welcome to Codecov 🎉

Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.

Thanks for integrating Codecov - We've got you covered ☂️

- Add Codecov upload step to unit-tests workflow with OIDC auth
- Configure workflow to run on push to main (establishes baseline)
- Add .codecov.yml with ignore patterns and flag configuration
- Use unit-tests flag to categorize coverage uploads

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
Signed-off-by: Mykhailo Kuznietsov <mkuznets@redhat.com>

@svor svor left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

lgtm

@openshift-ci

openshift-ci Bot commented Oct 27, 2026

Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: mkuznyetsov, svor

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@svor
svor merged commit afcaecb into main Oct 27, 2026
8 checks passed
@svor
svor deleted the add-codecov-coverage branch October 27, 2026 08:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants