CHANGED: CI triggers, release and docs#88
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. @@ Coverage Diff @@
## main #88 +/- ##
=======================================
Coverage 91.40% 91.40%
=======================================
Files 81 81
Lines 3746 3746
=======================================
Hits 3424 3424
Misses 322 322 🚀 New features to boost your workflow:
|
There was a problem hiding this comment.
Pull Request Overview
Align CI triggers for tests, publishing, and docs with the new release flow and bump package version.
- Bump package version to 0.7.4
- Enable Codecov coverage upload in tests
- Switch publish workflow to trigger on GitHub Releases; update docs workflow to run on main branch changes under docs/
Reviewed Changes
Copilot reviewed 4 out of 5 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| pyproject.toml | Bumps package version to 0.7.4 to match the new release. |
| .github/workflows/test.yml | Enables Codecov upload step post-coverage. |
| .github/workflows/publish.yml | Changes trigger from tag pushes to release published events; minor step name tweak. |
| .github/workflows/docs.yml | Changes trigger to push on main constrained to docs/** path. |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
| env: | ||
| CODECOV_TOKEN: ${{ secrets.CODECOV_TOKEN }} |
There was a problem hiding this comment.
[nitpick] Prefer passing the Codecov token via the action's input rather than as an environment variable to align with the action interface and reduce exposure to child steps. Example: add with: token: ${{ secrets.CODECOV_TOKEN }} and drop the env block.
| env: | |
| CODECOV_TOKEN: ${{ secrets.CODECOV_TOKEN }} | |
| token: ${{ secrets.CODECOV_TOKEN }} |
| branches: | ||
| - main | ||
| paths: | ||
| - 'docs/**' |
There was a problem hiding this comment.
[nitpick] If your docs build depends on MkDocs or top-level config files, consider including them in the path filter so changes to configuration also trigger the docs workflow (e.g., mkdocs.yml, README.md).
| - 'docs/**' | |
| - 'docs/**' | |
| - 'mkdocs.yml' | |
| - 'README.md' |
No description provided.