-
Notifications
You must be signed in to change notification settings - Fork 0
ci: Fix main coverage report on Codecov #111
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
Conversation
There was a problem hiding this 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 fixes the main coverage report on Codecov by updating the CI workflow trigger configuration and refactors the deployment documentation to streamline the release process.
- Added push trigger for the main branch to ensure coverage reports are properly updated on Codecov
- Simplified and restructured the deployment documentation in both README files
- Removed outdated TODO comments and complex release branch workflow
Reviewed Changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| .github/workflows/main.yml | Added push trigger for main branch to fix Codecov coverage reporting |
| README.md | Streamlined deployment documentation by removing complex release branch workflow |
| docs/soldoc/src/README.md | Updated deployment instructions to match the main README changes |
Comments suppressed due to low confidence (1)
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #111 +/- ##
=======================================
Coverage ? 83.92%
=======================================
Files ? 4
Lines ? 112
Branches ? 7
=======================================
Hits ? 94
Misses ? 17
Partials ? 1 ☔ View full report in Codecov by Sentry. |
Le-Caignec
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
No description provided.