-
Notifications
You must be signed in to change notification settings - Fork 1
feanil/setup semantic release #17
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
feanil
commented
Jan 2, 2026
- build: Invert version fetching.
- build: Add minimal config needed for python-semantic-release
- build: Add a github workflow for python-semantic-release
- fiX: Correct working directory for semantic release.
87ecfc7 to
4274662
Compare
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #17 +/- ##
==========================================
+ Coverage 88.20% 88.23% +0.02%
==========================================
Files 13 13
Lines 407 408 +1
Branches 16 16
==========================================
+ Hits 359 360 +1
Misses 40 40
Partials 8 8
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
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 configures python-semantic-release for automated versioning and releases. It corrects the parameter name from working-directory to directory for python-semantic-release actions to ensure proper operation in the backend subdirectory.
- Changes parameter from
working-directorytodirectoryfor python-semantic-release actions - Updates two action invocations to use the correct parameter name
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
.github/workflows/release.yml
Outdated
| path: dist | ||
| if-no-files-found: error | ||
| working-directory: './backend' | ||
| directory: './backend' |
Copilot
AI
Jan 2, 2026
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.
The 'directory' parameter is not supported by the actions/upload-artifact@v4 action. This action does not have a 'directory' parameter. Instead, the 'path' parameter should be updated to include the directory prefix if needed (e.g., 'backend/dist' instead of 'dist'), or the directory context should be handled differently. This will cause the workflow to fail.
Fix the directory for the reusable plugin as well as for the pypi upload step.
4274662 to
44e469b
Compare
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
Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.