fix: replace remaining uv tool install refs with uv pip install
Follow-up to #1909 — the npm package now installs into a dedicated venv instead of using uv tool, but these references were missed. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
This commit is contained in:
parent
14dafe08fd
commit
ec302cc596
3 changed files with 6 additions and 6 deletions
|
|
@ -700,9 +700,9 @@ def get_js_codeflash_install_step(pkg_manager: JsPackageManager, *, is_dependenc
|
||||||
# Codeflash will be installed with other dependencies
|
# Codeflash will be installed with other dependencies
|
||||||
return ""
|
return ""
|
||||||
|
|
||||||
# Install codeflash via uv (Python + uv are set up in the workflow)
|
# Install codeflash via uv pip (Python + uv are set up in the workflow)
|
||||||
return """- name: 📥 Install Codeflash
|
return """- name: 📥 Install Codeflash
|
||||||
run: uv tool install codeflash"""
|
run: uv pip install codeflash"""
|
||||||
|
|
||||||
|
|
||||||
def get_js_codeflash_run_command(pkg_manager: JsPackageManager, *, is_dependency: bool) -> str:
|
def get_js_codeflash_run_command(pkg_manager: JsPackageManager, *, is_dependency: bool) -> str:
|
||||||
|
|
|
||||||
|
|
@ -76,7 +76,7 @@ bun add --dev codeflash
|
||||||
```bash
|
```bash
|
||||||
pip install codeflash
|
pip install codeflash
|
||||||
# or
|
# or
|
||||||
uv tool install codeflash
|
uv pip install codeflash
|
||||||
```
|
```
|
||||||
|
|
||||||
The Python CLI orchestrates the optimization pipeline, while the npm package provides the JavaScript runtime (test runners, serialization, reporters).
|
The Python CLI orchestrates the optimization pipeline, while the npm package provides the JavaScript runtime (test runners, serialization, reporters).
|
||||||
|
|
|
||||||
|
|
@ -131,13 +131,13 @@ class TestGetJsCodeflashInstallStep:
|
||||||
|
|
||||||
assert result == ""
|
assert result == ""
|
||||||
|
|
||||||
def test_uv_tool_install_when_not_dependency(self) -> None:
|
def test_uv_pip_install_when_not_dependency(self) -> None:
|
||||||
"""Should generate uv tool install when not a dependency, regardless of package manager."""
|
"""Should generate uv pip install when not a dependency, regardless of package manager."""
|
||||||
for pkg_manager in (JsPackageManager.NPM, JsPackageManager.YARN, JsPackageManager.PNPM, JsPackageManager.BUN):
|
for pkg_manager in (JsPackageManager.NPM, JsPackageManager.YARN, JsPackageManager.PNPM, JsPackageManager.BUN):
|
||||||
result = get_js_codeflash_install_step(pkg_manager, is_dependency=False)
|
result = get_js_codeflash_install_step(pkg_manager, is_dependency=False)
|
||||||
|
|
||||||
assert "Install Codeflash" in result
|
assert "Install Codeflash" in result
|
||||||
assert "uv tool install codeflash" in result
|
assert "uv pip install codeflash" in result
|
||||||
|
|
||||||
|
|
||||||
class TestGetJsCodeflashRunCommand:
|
class TestGetJsCodeflashRunCommand:
|
||||||
|
|
|
||||||
Loading…
Reference in a new issue