-
Notifications
You must be signed in to change notification settings - Fork 4
refactor: remove deprecated python version and move to uv #11
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
refactor: remove deprecated python version and move to uv #11
Conversation
279baa7 to
f4fb8e0
Compare
f4fb8e0 to
7aa0a9f
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
This PR migrates the project from Poetry to uv as the package manager and build backend, updates Python version support from 3.9-3.13 to 3.10-3.13, and modernizes type annotations using PEP-604 style (str | None instead of Optional[str]).
Key changes:
- Removed Poetry configuration and migrated to PEP-621 compliant pyproject.toml with uv_build backend
- Updated CI/CD workflows to use uv instead of Poetry with dependency caching enabled
- Modernized type annotations to use union operator syntax (PEP-604)
Reviewed changes
Copilot reviewed 8 out of 10 changed files in this pull request and generated 8 comments.
Show a summary per file
| File | Description |
|---|---|
| pyproject.toml | Converted from Poetry format to PEP-621 with uv_build backend, updated dependencies and Python version constraints |
| tests/conftest.py | Updated import to use collections.abc.AsyncGenerator instead of typing.AsyncGenerator |
| taskiq_psqlpy/result_backend.py | Replaced Optional[T] with T | None syntax for type annotations |
| poetry.lock | Removed Poetry lock file (1738 lines deleted) |
| LICENSE | Added MIT license file |
| README.md | Minor formatting adjustment to Poetry installation section |
| .pre-commit-config.yaml | Updated to use uv run instead of poetry run, updated hook versions |
| .github/workflows/test.yaml | Migrated to uv with caching, updated to test Python 3.10-3.13, updated action versions |
| .github/workflows/release.yaml | Migrated to uv for building and publishing packages |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot <[email protected]>
Issue: #10
Remove poetry.lock and rewrite pyproject.toml in a way that compatible with PEP-621;
Change poetry run to uv run in pre-commit config;
Change CI/CD pipelines. They should use python versions from 3.10 to 3.13 and uv. You can find example of pipelines here. Don't forget to check that cache for dependencies in tests is enabled)
Check that there is no mentions of poetry in docs / readme files.
Please also replace old-style type annotations in repository code using this two ruff rules.