Skip to content

chore: update codecov#141

Merged
rustatian merged 1 commit into
masterfrom
fix/codecov
Oct 4, 2025
Merged

chore: update codecov#141
rustatian merged 1 commit into
masterfrom
fix/codecov

Conversation

@rustatian

@rustatian rustatian commented Oct 4, 2025

Copy link
Copy Markdown
Member

Reason for This PR

  • Codecov does not work.

Description of Changes

  • Correct report send operation.

License Acceptance

By submitting this pull request, I confirm that my contribution is made under
the terms of the MIT license.

PR Checklist

[Author TODO: Meet these criteria.]
[Reviewer TODO: Verify that these criteria are met. Request changes if not]

  • All commits in this PR are signed (git commit -s).
  • The reason for this PR is clearly provided (issue no. or explanation).
  • The description of changes is clear and encompassing.
  • Any required documentation changes (code and docs) are included in this PR.
  • Any user-facing changes are mentioned in CHANGELOG.md.
  • All added/changed functionality is tested.

Summary by CodeRabbit

  • Chores
    • Updated continuous integration to process code coverage within the main workflow.
    • Generates a coverage summary during the run and uploads it to the reporting service.
    • Consolidates steps by removing the previous archival and separate upload jobs.
    • Simplifies maintenance by keeping coverage transformation and upload in one place.
    • No changes to user-facing features or behavior.

Signed-off-by: Valery Piashchynski <piashchynski.valery@gmail.com>
@rustatian
rustatian requested a review from Copilot October 4, 2025 13:34
@rustatian rustatian self-assigned this Oct 4, 2025
@coderabbitai

coderabbitai Bot commented Oct 4, 2025

Copy link
Copy Markdown
Contributor

Walkthrough

The Linux GitHub Actions workflow replaces an archival and separate Codecov upload with an inline transformation step that processes coverage from tests/coverage-ci, writes tests/coverage-ci/summary.txt, and uploads that path to Codecov within the same workflow.

Changes

Cohort / File(s) Summary
CI workflow updates
.github/workflows/linux.yml
Removed archive step and separate Codecov workflow/job. Added “Transform code coverage results” step to process tests/coverage-ci and write summary.txt. Updated Codecov upload to use tests/coverage-ci (including summary.txt).

Sequence Diagram(s)

sequenceDiagram
    autonumber
    actor Dev as Developer
    participant GH as GitHub Actions (linux.yml)
    participant Build as Build & Test
    participant Transform as Transform Coverage
    participant Codecov as Codecov Uploader

    Dev->>GH: Push/PR triggers workflow
    GH->>Build: Run tests and generate coverage (tests/coverage-ci)
    GH->>Transform: Process coverage and write summary.txt
    note right of Transform: Outputs tests/coverage-ci/summary.txt
    GH->>Codecov: Upload coverage from tests/coverage-ci
    Codecov-->>GH: Report upload result
Loading

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

I hop through YAML fields so spry,
Tidying steps as logs glide by—
No archives now, just swift transform,
A summary hat to keep it warm.
Then up it goes to Codecov skies,
A carrot of green in coverage highs. 🥕✨

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Title Check ✅ Passed The title concisely identifies the update to the Codecov integration, matching the primary change in the workflow without extraneous details.
Description Check ✅ Passed The pull request description follows the repository template by providing a clear reason for the change, a concise description of the update, a license acceptance statement, and a completed checklist that addresses all required items.
Docstring Coverage ✅ Passed No functions found in the changes. Docstring coverage check skipped.
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix/codecov

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 2bc9b6c and 1a3c414.

📒 Files selected for processing (1)
  • .github/workflows/linux.yml (1 hunks)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)
  • GitHub Check: JOBS plugin (Go stable, PHP 8.4, OS ubuntu-latest)
🔇 Additional comments (1)
.github/workflows/linux.yml (1)

75-85: Coverage summarization looks correct.

The transformation step cleanly rebuilds a single Go coverage profile and uploads the expected path to Codecov. Nicely done.


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 and usage tips.

Copilot AI left a comment

Copy link
Copy Markdown

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 fixes Codecov integration by consolidating the coverage report generation and upload into a single job instead of using separate jobs with artifact upload/download.

  • Removes the separate codecov job and integrates coverage upload directly into the test job
  • Eliminates the artifact upload/download step that was causing issues
  • Updates the file path in the codecov action to correctly reference the generated summary file

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@codecov

codecov Bot commented Oct 4, 2025

Copy link
Copy Markdown

Codecov Report

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

Additional details and impacted files
@@             Coverage Diff             @@
##           master     #141       +/-   ##
===========================================
+ Coverage        0   47.14%   +47.14%     
===========================================
  Files           0       10       +10     
  Lines           0     1754     +1754     
===========================================
+ Hits            0      827      +827     
- Misses          0      860      +860     
- Partials        0       67       +67     

☔ 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.

@rustatian
rustatian merged commit bdcbe90 into master Oct 4, 2025
8 checks passed
@rustatian
rustatian deleted the fix/codecov branch October 4, 2025 13:40
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