Skip to content

fix: strip module prefix from coverage paths for Codecov#11

Merged
Snider merged 1 commit intomainfrom
dev
Mar 21, 2026
Merged

fix: strip module prefix from coverage paths for Codecov#11
Snider merged 1 commit intomainfrom
dev

Conversation

@Snider
Copy link
Contributor

@Snider Snider commented Mar 21, 2026

Summary by CodeRabbit

Release Notes

  • Chores
    • Updated continuous integration workflow to improve code coverage artifact processing for coverage tracking integration.

Co-Authored-By: Virgil <virgil@lethean.io>
@Snider Snider merged commit df1576b into main Mar 21, 2026
1 check was pending
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Mar 21, 2026

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 3678e882-bbe9-4df6-8c7e-2751320d3256

📥 Commits

Reviewing files that changed from the base of the PR and between 76f8ae4 and 954cd71.

📒 Files selected for processing (1)
  • .github/workflows/ci.yml

📝 Walkthrough

Walkthrough

The CI workflow's test-and-coverage step is updated to post-process the generated coverage report, removing the module path prefix from coverage artifact data before upload to Codecov.

Changes

Cohort / File(s) Summary
CI coverage processing
.github/workflows/ci.yml
Modified test-and-coverage step to execute go test -coverprofile=coverage.out ./tests/... and subsequently remove the path prefix dappco.re/go/core/ from the coverage output using sed.

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

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.

1 participant