Skip to content

Update codecov workflow#130

Merged
andreibancioiu merged 8 commits intomainfrom
workflows-06-10
Jul 8, 2025
Merged

Update codecov workflow#130
andreibancioiu merged 8 commits intomainfrom
workflows-06-10

Conversation

@andreibancioiu
Copy link
Collaborator

No description provided.

@andreibancioiu andreibancioiu self-assigned this Jun 10, 2025
@codecov
Copy link

codecov bot commented Jun 10, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 61.70%. Comparing base (0ab9266) to head (d78f87c).
Report is 314 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #130      +/-   ##
==========================================
- Coverage   71.60%   61.70%   -9.91%     
==========================================
  Files          33       48      +15     
  Lines        1747     3854    +2107     
==========================================
+ Hits         1251     2378    +1127     
- Misses        411     1320     +909     
- Partials       85      156      +71     

☔ View full report in Codecov by Sentry.
📢 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.

This comment was marked as outdated.

@andreibancioiu andreibancioiu changed the title Adjust GH workflows Update codecov workflow Jul 8, 2025
@andreibancioiu andreibancioiu requested a review from Copilot July 8, 2025 06:38

This comment was marked as outdated.

@andreibancioiu andreibancioiu requested a review from Copilot July 8, 2025 06:47
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 replaces the legacy bash uploader with the official Codecov GitHub Action and configures it to fail CI on errors while specifying the coverage file and authentication token.

  • Replaced bash <(curl ...) upload step with codecov/codecov-action@v5
  • Added fail_ci_if_error, files, and token inputs
Comments suppressed due to low confidence (2)

.github/workflows/code-coverage.yml:36

  • Consider pinning to a specific patch version of the Codecov Action (e.g., @v5.3.1) instead of the broad @v5 tag to avoid unexpected breaking changes in future updates.
        uses: codecov/codecov-action@v5

.github/workflows/code-coverage.yml:35

  • [nitpick] Adding a brief comment above this step to note the migration from the legacy bash uploader to the official Codecov Action could help future maintainers understand the change rationale.
      - name: Upload coverage

@andreibancioiu andreibancioiu merged commit 4b218f5 into main Jul 8, 2025
6 of 7 checks passed
@andreibancioiu andreibancioiu deleted the workflows-06-10 branch July 8, 2025 07:01
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.

3 participants