build(test-asyncpg): Migrate from pip to uv#62
Conversation
Replace requirements.txt with pyproject.toml and update run.sh to use uv for dependency management and script execution. Refs PY-2431 Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 2 potential issues.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit fb6d66c. Configure here.
| # create and activate virtual environment | ||
| python -m venv .venv | ||
| source .venv/bin/activate | ||
|
|
There was a problem hiding this comment.
uv not on PATH after install
Medium Severity
The uv installer, when run via curl | sh, updates PATH only within its subshell. This prevents the main script from finding uv for uv run, causing "command not found" errors despite a successful installation.
Reviewed by Cursor Bugbot for commit fb6d66c. Configure here.
| [project] | ||
| name = "test-asyncpg" | ||
| version = "0" | ||
| requires-python = ">=3.12" |
There was a problem hiding this comment.
Python version conflicts with pin
Medium Severity
requires-python = ">=3.12" conflicts with the existing test-asyncpg/.python-version pin of 3.10, so uv run can refuse the interpreter or fail resolution where the old pip workflow still worked on 3.10.
Reviewed by Cursor Bugbot for commit fb6d66c. Configure here.
Summary
requirements.txtwithpyproject.tomlfor dependency managementrun.shto useuv runinstead of manual venv/pip workflowTest plan
./run.shand verify the app runs correctly🤖 Generated with Claude Code