Skip to content

fix(ci): Correct Codecov reporting and file discovery #68

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 3 commits into from
Jul 27, 2025

Conversation

RandomProgramm3r
Copy link
Owner

This commit resolves a critical issue where Codecov was reporting inaccurate coverage metrics due to missing source files. The CI workflow has been updated to ensure all relevant files are discovered and processed correctly.
Key changes:

  • The coverage run command now explicitly specifies the source directory (--source='.') and the target applications (business, core, user). This forces coverage to analyze all relevant files, not just those touched by tests, fixing the incorrect file and line counts in Codecov reports.
  • The path to the .coveragerc configuration file is now correctly specified, ensuring custom settings are applied.
  • The Codecov GitHub Action is now configured with the correct directory to locate the coverage.xml report.
  • The entire test and coverage generation command has been streamlined for better clarity and reliability.

This commit updates the CI workflow to explicitly set the `directory` for the Codecov action. This ensures the action correctly locates the `coverage.xml` file within the `promo_code` directory, resolving potential pathing issues during the upload process.
This commit refactors the test and coverage generation step in the CI workflow.

- The `cd` command is removed, and paths are handled directly in the `coverage` command for better clarity.
- The `--source` option is added to ensure coverage is measured for the intended `business`, `core`, and `user` apps.
@codecov-commenter
Copy link

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@RandomProgramm3r RandomProgramm3r merged commit e37432b into main Jul 27, 2025
4 checks passed
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