ci(workflow): update codecov action and improve coverage processing #30
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Member
Ovizro
commented
Jan 4, 2025
Ovizro
commented
- Upgrade codecov-action to v5 in GitHub Actions workflow
- Add job name for online test with Tinode Server
- Update coverage combine command to use correct file path
- Remove test dependency from build_dist target in Makefile
- Upgrade codecov-action to v5 in GitHub Actions workflow - Add job name for online test with Tinode Server - Update coverage combine command to use correct file path - Remove test dependency from build_dist target in Makefile
- Add `include-hidden-files: true` to coverage data uploads - Include hidden files in chatbot logs artifact - Update upload-coverage job to depend on both test and online-test jobs
- Update chatbot log artifact path to use singular 'log' instead of 'logs' - Remove unnecessary 'include-hidden-files' option for chatbot logs - Add 'merge-multiple' option for coverage data artifact
- Simplify artifact download path to 'artifacts' - Add step to display structure of downloaded files - Update coverage combine command to use 'artifacts/*'
- Change coverage file pattern from .coverage* to .coverage.* - Remove display of downloaded file structure - Simplify coverage combine command
- Add actions/checkout@v4 step to ensure proper repository checkout - This change improves the reliability of the test-cov workflow
Welcome to Codecov 🎉Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests. ℹ️ You can also turn on project coverage checks and project coverage reporting on Pull Request comment Thanks for integrating Codecov - We've got you covered ☂️ |
- Uncommented workflow_dispatch condition for online tests - Increased log retention days for artifacts - Combined coverage data collection steps - Updated test setup to include plugin enablement - Improved user and topic services - Refactored information retrieval methods
- Refactor get_sub function to handle subscription checking and caching more efficiently - Fix subscription retrieval for both topic and user subscriptions - Update tests to reflect new subscription behavior - Remove unnecessary code and improve code readability in various files
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.