-
Notifications
You must be signed in to change notification settings - Fork 6
Switch from Poetry to uv for package management #223
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
- Updated pyproject.toml to use standard PEP 621 format with hatchling build backend - Removed poetry-specific sections and converted to project.optional-dependencies - Generated uv.lock to replace poetry.lock - Updated GitHub Actions CI workflow to use uv commands - Updated release workflow to use uv build and publish commands Co-authored-by: norkans7 <1040571+norkans7@users.noreply.github.com>
- Removed isort from main dependencies (already in dev dependencies) - Added .venv/ to .gitignore for uv virtual environments Co-authored-by: norkans7 <1040571+norkans7@users.noreply.github.com>
.github/workflows/ci.yml
Outdated
| - name: Install Poetry | ||
| uses: snok/install-poetry@v1 | ||
| - name: Install uv | ||
| uses: astral-sh/setup-uv@v5 |
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.
@copilot use astral-sh/setup-uv@v7 for github action
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.
Updated to astral-sh/setup-uv@v7 in both the test and release jobs (c5fa260)
- Updated astral-sh/setup-uv from v5 to v7 in both test and release jobs Co-authored-by: norkans7 <1040571+norkans7@users.noreply.github.com>
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #223 +/- ##
=======================================
Coverage 93.32% 93.32%
=======================================
Files 29 29
Lines 1843 1843
=======================================
Hits 1720 1720
Misses 123 123 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
@copilot rebase this from the main branch and update if needed |
Rebased the PR on the latest main branch (f89c8d7). The rebase included:
The changes are in commit 7eab4c2. Note: Due to authentication limitations, the force push is pending. The rebase is complete locally and all tests pass. |
|
@copilot try again |
|
@copilot close this PR and restart a new one from main branch |
I don't have the ability to close PRs or create new branches through the automated system. However, I can help prepare the changes on a fresh branch based on main. If you'd like, you can:
Alternatively, the rebased changes in commit b5a6cdf are ready and just need a force push. If you have git push access, you can run: |
Switch to uv Python Package Manager
This PR switches the project from Poetry to uv for package management.
Implementation Plan:
Changes Made:
pyproject.tomlfrom Poetry format to standard PEP 621 formatpoetry-coretohatchlingtool.poetry.group.dev.dependenciestoproject.optional-dependenciesisortdependency (only kept in dev dependencies)uv.lockfile (replacespoetry.lock).github/workflows/ci.ymlto useuvcommands instead ofpoetryastral-sh/setup-uvaction to v7 for latest features and fixes.venv/to.gitignorefor uv virtual environmentsBenefits:
Testing:
✅ Code checks pass (
uv run python ./code_check.py)✅ Package builds successfully (
uv build)✅ No security vulnerabilities found (CodeQL)
✅ Code review completed
✅ Rebased on latest main with urllib3 2.6.3
Original prompt
💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.