Skip to content

build(test-potel-trace-propagation): Migrate backend from pip to uv#59

Merged
ericapisani merged 1 commit into
mainfrom
PY-2486-test-potel-trace-propagation
May 21, 2026
Merged

build(test-potel-trace-propagation): Migrate backend from pip to uv#59
ericapisani merged 1 commit into
mainfrom
PY-2486-test-potel-trace-propagation

Conversation

@ericapisani

Copy link
Copy Markdown
Member

Summary

  • Replace backend/requirements.txt with backend/pyproject.toml for dependency management
  • Update backend/run.sh to use uv run instead of manual venv/pip workflow
  • Only the backend subdirectory is Python; frontend/webservice are unaffected
  • Part of PY-2428 migration effort

Test plan

  • Run cd backend && ./run.sh and verify the Flask app starts correctly

🤖 Generated with Claude Code

Replace backend/requirements.txt with backend/pyproject.toml and update
backend/run.sh to use uv for dependency management and script execution.

Refs PY-2486
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@linear-code

linear-code Bot commented May 21, 2026

Copy link
Copy Markdown

PY-2486

PY-2428

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 2b74e47. Configure here.

# create and activate virtual environment
python -m venv .venv
source .venv/bin/activate

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Installed uv not in PATH for current session

High Severity

When uv is not installed and the curl | sh installer runs, it modifies shell profile files (.bashrc, .zshrc, etc.) to add uv to PATH. Those profile changes only take effect in new shell sessions, not the current one. So when execution reaches uv run flask --app main run immediately after, the shell still can't find uv and the script fails with a "command not found" error.

Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit 2b74e47. Configure here.

@ericapisani ericapisani merged commit c5d4814 into main May 21, 2026
11 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.

1 participant