-
Notifications
You must be signed in to change notification settings - Fork 2
chore: add uv for python sdk #33
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
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.
Just left some remarks and questions, but we're pretty much there. 🙂
Have you done any test of any kind, to validate your changes? This will break the current release workflow I guess, so we'll have to tackle your other PR after this one (at least before any new fix/feature)
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.
Two very small remarks to polish things up 😇
[tool.hatch.build.targets.wheel] Is used because the name of the package is just linkup and the name in the pyproject is "linkup_sdk" |
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.
All good! ✌️
Description
This PR replaces 'pip' with 'uv' for managing development dependencies.
These changes introduce a faster and more reproducible development environment using
uv
.Checklist
make install-dev
andmake test
README.md
if my changes affected itlinkup/_version.py
if I plan on creating a new release