Skip to content

Use UV for dependency management#45

Merged
tudoramariei merged 3 commits intodanniel/reReDockerizefrom
tamariei/uv-dockerize
Jun 13, 2025
Merged

Use UV for dependency management#45
tudoramariei merged 3 commits intodanniel/reReDockerizefrom
tamariei/uv-dockerize

Conversation

@tudoramariei
Copy link
Copy Markdown
Member

  • update dockerfiles, makefiles, pyproject
  • remove requirements files in favour of uv.lock & pyproject

- update dockerfiles, makefiles, pyproject
- remove requirements files in favour of uv.lock & pyproject
@tudoramariei tudoramariei requested review from Copilot and danniel June 12, 2025 12:11
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

Migrate dependency management from pip-compiled requirements to Astral UV by updating Dockerfiles, Makefile, and pyproject.toml and removing old requirements files.

  • Added UV installer and replaced virtualenv creation and pip installs with uv venv and uv sync
  • Removed all requirements.txt/.in files, introduced pyproject.toml + uv.lock
  • Updated Makefile targets to use uv sync instead of pip-compile

Reviewed Changes

Copilot reviewed 9 out of 9 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
docker/dockerfiles/Dockerfile.dev Install UV, swap python3 -m venv + pip installs for uv venv + uv sync
docker/dockerfiles/Dockerfile Add UV installer, replace pip venv/install with uv venv + uv sync
backend/requirements.txt Removed obsolete requirements file
backend/requirements.in Removed obsolete requirements file
backend/requirements-dev.txt Removed obsolete dev requirements file
backend/requirements-dev.in Removed obsolete dev requirements file
backend/pyproject.toml Added project metadata, dependencies, dev groups; removed pip-tools config
Makefile Changed dependency targets to use uv sync and updated comments accordingly
Comments suppressed due to low confidence (4)

backend/pyproject.toml:2

  • Project name is empty; specify a valid package name to ensure correct metadata and distribution.
name = ""

backend/pyproject.toml:120

  • Black's target-version is set to py312 but requires-python is >=3.13; update to py313 (or appropriate) for consistency.
target-version = ["py312"]

Makefile:154

  • The comment mentions pip compile but the target uses uv sync; update the description to accurately reflect the new command.
requirements-build:               ## run uv pip compile and add requirements from the *.in files

Makefile:160

  • Clarify this comment to reflect use of uv sync --active -U instead of pip-compile for updating dependencies and lock file.
requirements-update:              ## run uv pip compile and rebuild the requirements files

Comment thread docker/dockerfiles/Dockerfile.dev
Comment thread docker/dockerfiles/Dockerfile Outdated
Comment thread docker/dockerfiles/Dockerfile Outdated
Comment thread docker/dockerfiles/Dockerfile Outdated
@tudoramariei tudoramariei force-pushed the tamariei/uv-dockerize branch from 4368ff0 to 7871506 Compare June 13, 2025 07:41
@tudoramariei tudoramariei merged commit e6c8ac2 into danniel/reReDockerize Jun 13, 2025
1 check passed
@tudoramariei tudoramariei deleted the tamariei/uv-dockerize branch June 13, 2025 07:59
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.

3 participants