Skip to content
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

Replace pip with uv for the project files #5356

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

foarsitter
Copy link
Collaborator

Description

uv is maturing out and I played around a bit by replacing pip with uv for the project files. The generated project template stay's untouched.

Rationale

Dependencies

The dependencies located in requirements.txt are now included in pyproject.toml in the project.dependencies section. pyproject-fmt sorts them alphabetical so I had to remove the comments. There is also a lock file (uv.lock) but dependabot doesn't have support for this at the moment.

Actions

Each action uses astral-sh/setup-uv@v2 for installing and caching the dependencies. From what I saw the actions are a few seconds faster but not much.

Scripts

The scripts located in /scripts/ have now a dependencies section for standalone execution: https://docs.astral.sh/uv/guides/scripts/ and are run by invoking uv run scripts/create_django_issue.py

Looking forward to implement uv in the project template to!

@browniebroke
Copy link
Member

Sounds cool!

There is also a lock file (uv.lock) but dependabot doesn't have support for this at the moment.

Found the issue tracking that dependabot/dependabot-core#10478

.github/workflows/ci.yml Show resolved Hide resolved
pyproject.toml Show resolved Hide resolved
scripts/create_django_issue.py Outdated Show resolved Hide resolved
@browniebroke
Copy link
Member

Alternatively, renovate already has uv support it seems: renovatebot/renovate#30756

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants