Skip to content

Conversation

@JasonXuDeveloper
Copy link
Owner

Summary

  • Add fixes section to codecov.yml that strips the Docker container path prefix (/github/workspace/) from coverage reports
  • Unity test runner generates paths like /github/workspace/UnityProject/... inside the Docker container
  • Codecov expects paths relative to repo root: UnityProject/...
  • The path mismatch was causing reports to be marked as "unusable"

Test plan

  • Wait for PR tests to run (note: this config-only change won't trigger Unity tests due to path filters)
  • Merge this PR and verify coverage uploads succeed on subsequent PRs that modify package code
  • Check Codecov dashboard shows actual coverage percentages instead of "unknown"

🤖 Generated with Claude Code

Add fixes section to strip Docker container path prefix (/github/workspace/)
from coverage reports generated by Unity test runner, allowing Codecov to
correctly match source files in the repository.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Signed-off-by: JasonXuDeveloper - 傑 <jason@xgamedev.net>
Copilot AI review requested due to automatic review settings January 30, 2026 01:58
@JasonXuDeveloper JasonXuDeveloper enabled auto-merge (squash) January 30, 2026 01:59
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

Updates Codecov configuration to correct file path prefixes in uploaded coverage reports generated inside the GitHub Actions Docker workspace, preventing Codecov from marking reports as “unusable”.

Changes:

  • Add a Codecov fixes rule to strip the /github/workspace/ prefix from coverage file paths.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Auto-approved: Copilot review found no issues and Unity Tests passed (or were skipped for non-code changes).

@JasonXuDeveloper JasonXuDeveloper merged commit aa49bc3 into master Jan 30, 2026
21 checks passed
@JasonXuDeveloper JasonXuDeveloper deleted the fix/codecov-path-mismatch branch January 30, 2026 02:26
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