Skip to content

Replace pip with uv #579

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.

Sign up for GitHub

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

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

Replace pip with uv #579

Changes from 1 commit
Commits
Show all changes
9 commits
Select commit Hold shift + click to select a range
ef1ed5c
Replace `pip` with `uv`
benmezger Aug 30, 2025
456eca8
Migrate Ruff settings
benmezger Aug 30, 2025
ee8b5ea
Refactor Github action to use Ruff
benmezger Aug 30, 2025
985c4f5
Apply ruff check changes
benmezger Aug 30, 2025
617ffad
Ensure Makefile runs with UV
benmezger Aug 30, 2025
cf73476
Add SMTP group
benmezger Aug 30, 2025
56e3f32
Update README
benmezger Aug 30, 2025
990f824
Update Gitlab CI configuration
benmezger Aug 30, 2025
6634c9b
Remove `.python-version` and use toml instead
benmezger Aug 30, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 3 additions & 3 deletions Makefile
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -18,15 +18,15 @@ test-pdb: test
.PHONY: test lint coverage

lint:
flake8 --extend-exclude "*/migrations/,local_settings.py" devel main mirrors news packages releng templates todolists visualize *.py
uv run flake8 --extend-exclude "*/migrations/,local_settings.py" devel main mirrors news packages releng templates todolists visualize *.py
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

should re-evaluate if ruff already has the formatting rules, or ugh ruff format I guess.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ruff actually replaces flake8, so we can technically get rid of this and have one tool to handle formatting/linting.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

"replaces", not entirely most of the E1 rules are not yet implemented and if implemented are preview rules.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Anyway, this will be handled in a follow up by moving to ruff format


collectstatic:
python manage.py collectstatic --noinput
uv run python manage.py collectstatic --noinput

test: test-py

test-py coverage:
DJANGO_SETTINGS_MODULE=settings ${PYTEST} ${PYTEST_OPTIONS} ${PYTEST_COVERAGE_OPTIONS} ${PYTEST_INPUT}
DJANGO_SETTINGS_MODULE=settings uv run ${PYTEST} ${PYTEST_OPTIONS} ${PYTEST_COVERAGE_OPTIONS} ${PYTEST_INPUT}

open-coverage: coverage
${BROWSER} htmlcov/index.html