Skip to content

Migrate from codecov#130

Merged
AnkushinDaniil merged 5 commits intomainfrom
daniil/update-code-coverage
Aug 20, 2025
Merged

Migrate from codecov#130
AnkushinDaniil merged 5 commits intomainfrom
daniil/update-code-coverage

Conversation

@AnkushinDaniil
Copy link
Collaborator

No description provided.

@AnkushinDaniil AnkushinDaniil force-pushed the daniil/update-code-coverage branch 10 times, most recently from 596ba5d to 04c3389 Compare August 8, 2025 09:12
@AnkushinDaniil AnkushinDaniil changed the title update code coverage workflow Migrate from codecov Aug 8, 2025
@AnkushinDaniil AnkushinDaniil marked this pull request as ready for review August 8, 2025 10:51
Copilot AI review requested due to automatic review settings August 8, 2025 10:51
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR migrates from Codecov to a self-hosted coverage solution using Coverlet.MSBuild and GitHub Actions, implementing coverage reporting directly within the CI workflow.

  • Removes Codecov configuration and external service dependency
  • Implements inline coverage collection using Coverlet.MSBuild package
  • Adds automated PR comments and coverage threshold validation (60% minimum)

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.

File Description
codecov.yml Complete removal of Codecov configuration file
.github/workflows/ci.yml Migration to self-hosted coverage with Coverlet integration, PR commenting, and threshold validation

Copy link
Contributor

@wurdum wurdum left a comment

Choose a reason for hiding this comment

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

This coverage report takes up too much space in PR and creates a lot of noise.
Can we have the coverage report message to show only very basic info (coverage percentage, how it's changed, and whether it passes the threshold), and the link to the full report?

@NethermindEth NethermindEth deleted a comment from github-actions bot Aug 12, 2025
@AnkushinDaniil AnkushinDaniil force-pushed the daniil/update-code-coverage branch from 2bfe056 to a764df9 Compare August 20, 2025 12:05
@github-actions
Copy link
Contributor

Code Coverage

Package Line Rate Branch Rate Health
Nethermind.Arbitrum 78% 54%
Summary 78% (4750 / 6114) 54% (1308 / 2419)

Minimum allowed line rate is 60%

@AnkushinDaniil AnkushinDaniil merged commit 449dbbb into main Aug 20, 2025
5 checks passed
@AnkushinDaniil AnkushinDaniil deleted the daniil/update-code-coverage branch August 20, 2025 12:14
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.

5 participants