Migrate from poetry to uv for dependency and package management#986
Merged
sdb9696 merged 5 commits intopython-kasa:masterfrom Sep 6, 2024
Merged
Migrate from poetry to uv for dependency and package management#986sdb9696 merged 5 commits intopython-kasa:masterfrom
sdb9696 merged 5 commits intopython-kasa:masterfrom
Conversation
This was referenced Jun 18, 2024
a6ff8d0 to
914a54b
Compare
914a54b to
e009300
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #986 +/- ##
=======================================
Coverage 92.55% 92.55%
=======================================
Files 96 96
Lines 6100 6100
Branches 1504 1504
=======================================
Hits 5646 5646
Misses 354 354
Partials 100 100 ☔ View full report in Codecov by Sentry. |
rytilahti
reviewed
Sep 6, 2024
Member
There was a problem hiding this comment.
A couple of quick comments. For some reason I need to manually define -p python for uv commands, or it fails otherwise like this:
uv sync --all-extras
error: No interpreter found for executable name `kasa37` in managed installations or system path
Collaborator
Author
Which version of uv are you using? I have been using 0.4.5 installed via |
Merged
sdb9696
added a commit
that referenced
this pull request
Sep 10, 2024
## [0.7.3](https://github.com/python-kasa/python-kasa/tree/0.7.3) (2024-09-10) [Full Changelog](0.7.2...0.7.3) **Release summary:** - Migrate from `poetry` to `uv` for package/project management. - Various minor code improvements **Project maintenance:** - Do not regenerate aes key pair [\#1114](#1114) (@sdb9696) - Fix tests due to yarl URL str output change [\#1112](#1112) (@sdb9696) - Add missing type hints to alarm module [\#1111](#1111) (@rytilahti) - Add KH100 EU fixtures [\#1109](#1109) (@rytilahti) - Migrate from poetry to uv for dependency and package management [\#986](#986) (@sdb9696)
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an acco
67EF
unt?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
As of uv 0.3 there is now official support for package management/projects.
To use the environment (
.venv) simply run:Obviously the speed is great but also the support for downloading and using multiple python versions is really great.
Notes:
MANIFEST.into be added.poetry versionto update the project version so theRELEASING.mdstep usessed. (hatchhas some support for this but I think it's better to keep it simple for now.)uv runworks fine so was able to drop therun-in-env.sh scriptrequired for a couple of pre-commit checks. (This may have actually worked with poetry too although I seem to recall it had problems in the CI on windows)uv.lockfile.yarl>=1.9.5released since the 31st August break our tests as thestr(URL)now omits port 80 so I've fixed those tests.Replaces #985