-
Notifications
You must be signed in to change notification settings - Fork 0
automate releases and fix linux workflow #1
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
Signed-off-by: Rahul Krishna <[email protected]>
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 automates the release process and fixes compatibility issues on Linux by updating changelog entries, adding a release configuration for changelog generation, and introducing a GitHub Actions workflow for building, testing, and publishing.
- Updated CHANGELOG with new features, fixes, and documentation details.
- Added
.github/workflows/release_config.jsonto categorize changelog entries. - Introduced
.github/workflows/release.ymlto automate tagging, testing, and publishing to GitHub and PyPI.
Reviewed Changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| CHANGELOG.md | Expanded “Added”/“Changed”/“Fixed” sections for new behaviors |
| .github/workflows/release_config.json | New config for release-changelog-builder-action |
| .github/workflows/release.yml | New GitHub Actions workflow for end-to-end releases |
Comments suppressed due to low confidence (2)
CHANGELOG.md:13
- This entry is duplicated by the 'Updated README.md with detailed prerequisite packages…' under 'Changed'; consider consolidating or clarifying their differences to avoid confusion.
- Comprehensive system package requirements documentation in README.md
.github/workflows/release.yml:1
- [nitpick] For consistency with the tool's branding, consider capitalizing ‘uv’ to ‘UV’ in the workflow name.
name: Python uv Release
| "title": "## \uD83D\uDCDA Documentation", | ||
| "labels": [ | ||
| "documentation", | ||
| "doc" | ||
| ] | ||
| }, | ||
| { | ||
| "title": "## \uD83D\uDEA6 Tests", | ||
| "labels": [ | ||
| "test" | ||
| ] | ||
| }, | ||
| { | ||
| "title": "## \uD83D\uDEE0 Other Updates", |
Copilot
AI
Jul 11, 2025
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.
[nitpick] Using Unicode escape sequences in the JSON reduces readability; consider using the actual emoji character (📚) for clarity and easier editing.
| "title": "## \uD83D\uDCDA Documentation", | |
| "labels": [ | |
| "documentation", | |
| "doc" | |
| ] | |
| }, | |
| { | |
| "title": "## \uD83D\uDEA6 Tests", | |
| "labels": [ | |
| "test" | |
| ] | |
| }, | |
| { | |
| "title": "## \uD83D\uDEE0 Other Updates", | |
| "title": "## 📚 Documentation", | |
| "labels": [ | |
| "documentation", | |
| "doc" | |
| ] | |
| }, | |
| { | |
| "title": "## 🚦 Tests", | |
| "labels": [ | |
| "test" | |
| ] | |
| }, | |
| { | |
| "title": "## 🛠 Other Updates", |
|
|
||
| - name: Install uv | ||
| run: | | ||
| curl -LsSf https://astral.sh/uv/install.sh | sh |
Copilot
AI
Jul 11, 2025
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.
Piping a remote script directly into shell can pose a security risk; consider pinning to a known version, verifying a checksum, or downloading the script separately for review before execution.
| curl -LsSf https://astral.sh/uv/install.sh | sh | |
| curl -LsSf -o install.sh https://astral.sh/uv/install.sh | |
| echo "Expected checksum: <INSERT_CHECKSUM_HERE>" | |
| echo "<INSERT_CHECKSUM_HERE> install.sh" | sha256sum --check --status | |
| if [ $? -ne 0 ]; then | |
| echo "Checksum verification failed! Exiting..." | |
| exit 1 | |
| fi | |
| sh install.sh |
Summary
Enhanced CLI error handling, improved system compatibility, and added comprehensive documentation for system requirements. This release focuses on robustness and better user experience across different Python environments and operating systems.
Motivation and Context
Users were experiencing issues when:
These improvements ensure the tool works reliably across different environments and provides clear feedback when issues occur.
How Has This Been Tested?
Breaking Changes
--quietflag replaced with-v(verbose) flag in test casesTypes of changes
Checklist
Additional context
Key Improvements:
Files Modified: