Skip to content

Add uv setup prerequisites to README run/test workflow#3

Open
phuocphn with Copilot wants to merge 2 commits into
dev/switch-to-uvfrom
copilot/fix-code-as-per-review-comment
Open

Add uv setup prerequisites to README run/test workflow#3
phuocphn with Copilot wants to merge 2 commits into
dev/switch-to-uvfrom
copilot/fix-code-as-per-review-comment

Conversation

Copilot AI commented Jun 1, 2026

Copy link
Copy Markdown

The linked review comment flagged that README usage moved to uv run ... without telling contributors how to install uv or create a synced environment first. This PR addresses only that gap with minimal documentation changes.

  • Setup prerequisites

    • Added a new ## Setup section in README.md with explicit environment bootstrap commands:
      • install uv
      • sync dependencies via uv sync
  • Execution docs alignment

    • Kept existing Run and Test commands intact, now preceded by the required setup context so they are actionable in a fresh clone.
python -m pip install uv
uv sync

Copilot AI changed the title [WIP] Fix code based on review comment Add uv setup prerequisites to README run/test workflow Jun 1, 2026
Copilot AI requested a review from phuocphn June 1, 2026 08:05
@phuocphn

phuocphn commented Jun 1, 2026

Copy link
Copy Markdown
Contributor

@copilot Please refer users to the official uv installation:
Installation | uvhttps://docs.astral.sh/uv/getting-started/installation/

@phuocphn
phuocphn marked this pull request as ready for review June 1, 2026 08:07
@phuocphn

phuocphn commented Jun 1, 2026

Copy link
Copy Markdown
Contributor

@copilot Please refer users to the official uv installation:
Installation | uvhttps://docs.astral.sh/uv/getting-started/installation/

@phuocphn

phuocphn commented Jun 1, 2026

Copy link
Copy Markdown
Contributor

@copilot : Please refer users to the official uv installation:
Installation | uvhttps://docs.astral.sh/uv/getting-started/installation/

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.

2 participants