Skip to content

Fix issue 458 -- move to uv from pip#462

Merged
benadida merged 1 commit intomasterfrom
claude/fix-issue-458-kraBQ
Jan 17, 2026
Merged

Fix issue 458 -- move to uv from pip#462
benadida merged 1 commit intomasterfrom
claude/fix-issue-458-kraBQ

Conversation

@benadida
Copy link
Owner

  • Replace requirements.txt with pyproject.toml for modern dependency management
  • Add uv.lock for reproducible builds
  • Update INSTALL.md with uv installation instructions
  • Update CLAUDE.md commands to use uv run
  • Add .venv to .gitignore for uv's virtual environment

Closes #458

Copy link
Contributor

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

This PR migrates the project from pip-based dependency management to uv, a modern Python package manager. The migration involves replacing requirements.txt with pyproject.toml, adding a lock file for reproducible builds, and updating documentation.

Changes:

  • Replaced requirements.txt with pyproject.toml and uv.lock for modern dependency management
  • Updated Python version requirement from 3.12 to 3.13
  • Updated installation and usage documentation to use uv commands

Reviewed changes

Copilot reviewed 3 out of 6 changed files in this pull request and generated 5 comments.

Show a summary per file
File Description
pyproject.toml Added new project configuration with dependencies, development dependencies, and metadata
uv.lock Added lock file for reproducible dependency resolution
requirements.txt Removed old pip requirements file
INSTALL.md Updated installation instructions to use uv instead of pip/venv
CLAUDE.md Updated command examples to use "uv run" prefix
.gitignore Added .venv directory to ignore uv's virtual environment

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

pyproject.toml Outdated
"python-dateutil>=2.9.0",
"bleach>=6.1.0",
"django-csp>=3.8",
"validate_email>=1.3",
Copy link

Copilot AI Jan 17, 2026

Choose a reason for hiding this comment

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

The dependency name should be "validate-email" (with a hyphen) to match the package name on PyPI, not "validate_email" (with an underscore). While Python often accepts both forms, using the canonical PyPI name is the correct approach.

Suggested change
"validate_email>=1.3",
"validate-email>=1.3",

Copilot uses AI. Check for mistakes.
pyproject.toml Outdated
license = "Apache-2.0"
requires-python = ">=3.13"
dependencies = [
"django>=5.2.9",
Copy link

Copilot AI Jan 17, 2026

Choose a reason for hiding this comment

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

The lock file specifies Django 6.0.1, but pyproject.toml requires "django>=5.2.9". This means newer major versions (6.x) are allowed, which could be a breaking change. If the project has only been tested with Django 5.x, consider pinning the major version with "django>=5.2.9,<6" to prevent unexpected upgrades.

Suggested change
"django>=5.2.9",
"django>=5.2.9,<6",

Copilot uses AI. Check for mistakes.
@benadida benadida force-pushed the claude/fix-issue-458-kraBQ branch 2 times, most recently from aa045f3 to ed1e999 Compare January 17, 2026 18:34
- Replace requirements.txt with pyproject.toml for modern dependency management
- Add uv.lock for reproducible builds
- Update INSTALL.md with uv installation instructions
- Update CLAUDE.md commands to use uv run
- Add .venv to .gitignore for uv's virtual environment

Closes #458
@benadida benadida force-pushed the claude/fix-issue-458-kraBQ branch from ed1e999 to a9246db Compare January 17, 2026 18:36
@benadida benadida merged commit 604fac4 into master Jan 17, 2026
3 checks passed
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.

migrate from pip to uv

3 participants