build: switch test setup from pip to uv#323
Conversation
There was a problem hiding this comment.
Pull request overview
This PR migrates local test scripts and the GitHub Actions pytest workflow from pip to uv, aligning test/developer setup around pyproject.toml extras and standardizing commands across environments.
Changes:
- Switch
run_tests.sh/run_tests.ps1touv pip installanduv run pytest. - Update the GitHub Actions pytest workflow to use
astral-sh/setup-uvand run tests viauv. - Relax
[tool.uv].required-versionand update README installation/testing examples touv.
Reviewed changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated 4 comments.
Show a summary per file
| File | Description |
|---|---|
run_tests.sh |
Uses uv for editable install and running pytest. |
run_tests.ps1 |
Uses uv for editable install and running pytest on Windows. |
pyproject.toml |
Relaxes minimum uv version requirement. |
README.MD |
Updates local install/test/docs to uv-based commands. |
.github/workflows/pytest.yml |
Switches CI setup from setup-python/pip to setup-uv/uv. |
6008a4f to
9568f03
Compare
|
I addressed the actionable ones:
I left the workflow indentation unchanged, since the YAML there is valid as-is and the workflow was already parsing/running correctly. |
9568f03 to
1d2152e
Compare
|
Looks good, let’s give the copilot another round |
|
@filiplajszczak Sorry to come back to this with run_tests.sh I just setup a new environment. When I run run_tests.sh twice, I get: This is a problem, because the script is intended to run without manual interaction. I would set UV_VENV_CLEAR=1 to skip the prompt. It seems uv fullfills the needed packages based on a cahce. What do you suggest? |
Good catch. I’d use
|
1d2152e to
76a26d0
Compare
MaStr
left a comment
There was a problem hiding this comment.
That looks good, thank your very much!
Switch the
pytestworkflow and test helper scripts frompiptouv, update the README examples, and relax the minimumuvversion.Leave the
Dockerfileunchanged for now; container build migration can follow separately.