build(test-celery-max-tasks-per-child): Migrate from pip to uv#65
Conversation
Replace requirements.txt with pyproject.toml and update run.sh and run-celery.sh to use uv for dependency management and script execution. Refs PY-2434 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 c322f0a. Configure here.
| reset | ||
|
|
||
| # create and activate virtual environment | ||
| python -m venv .venv |
There was a problem hiding this comment.
uv missing after install
Medium Severity
When uv is installed via curl | sh, the installer doesn't update the current shell's PATH. This causes the immediate uv run command to fail with "command not found" in the same script execution.
Reviewed by Cursor Bugbot for commit c322f0a. Configure here.
| [project] | ||
| name = "test-celery-max-tasks-per-child" | ||
| version = "0" | ||
| requires-python = ">=3.12" |
There was a problem hiding this comment.
Python version constraint mismatch
Medium Severity
The pyproject.toml specifies requires-python = ">=3.12", but the .python-version file in this directory is pinned to 3.10. This conflict can cause uv sync or uv run to fail or select an incorrect Python interpreter, unlike other similar projects where these versions are aligned.
Reviewed by Cursor Bugbot for commit c322f0a. Configure here.
Summary
requirements.txtwithpyproject.tomlfor dependency managementrun.shandrun-celery.shto useuv runinstead of manual venv/pip workflowTest plan
./run-celery.shand verify the celery worker starts./run.shand verify the app runs correctly🤖 Generated with Claude Code