Merged
Conversation
Contributor
There was a problem hiding this comment.
PR Overview
This PR refactors the project to transition from Poetry to uv for dependency management, project installation, and test execution. Key changes include:
- Updating pyproject.toml to use the new [project] configuration and uv settings with Hatchling.
- Modifying GitHub workflows to install uv, set up Python, and run tests and commands using uv.
- Revising documentation and README links and commands to reflect the switch from Poetry to uv.
Reviewed Changes
| File | Description |
|---|---|
| pyproject.toml | Refactors configuration from Poetry to uv and Hatchling |
| .github/workflows/cat-test-examples.yml | Updates steps to install uv and run tests using uv |
| .github/workflows/python-tests.yml | Similar update to install uv and run tests commands |
| examples/team_recommender/readme.md | Updates local setup instructions to use uv |
| docs/getting-started.md | Adds instructions to use uv for project setup |
| README.md | Switches test and code quality commands to uv and updates documentation links |
Copilot reviewed 8 out of 8 changed files in this pull request and generated no comments.
Comments suppressed due to low confidence (2)
.github/workflows/cat-test-examples.yml:35
- The test command now targets a specific subdirectory which differs from the previous broader directory. Please verify that the new path is correct and intentional for running the example tests.
run: uv run pytest examples/team_recommender/tests/example_4_gate_on_success_threshold/
README.md:37
- The archive link has been updated from pointing to 'wiki/ARCHIVE.md' to 'wiki'. Please confirm that this change correctly reflects the new location of the archive content.
- [Archive - To see potentially helpful previous iterations of the code](wiki)
tkersey
force-pushed
the
transition-to-uv
branch
from
6e23129 to
f0f4f43
Compare
Contributor
There was a problem hiding this comment.
PR Overview
This PR transitions the project’s workflows and configuration from Poetry to uv, updating the setup, installation, and test commands accordingly. Key changes include:
- Refactoring the pyproject.toml to use the [project] section with dependency groups instead of Poetry’s configuration.
- Updating GitHub workflows to install and run the project using uv instead of Poetry.
- Modifying documentation and example files to reflect the new uv-based setup.
Reviewed Changes
| File | Description |
|---|---|
| pyproject.toml | Refactored project configuration and dependency groups. |
| .github/workflows/cat-test-examples.yml | Replaced Poetry steps with uv installation and command usage. |
| .github/workflows/python-tests.yml | Updated test workflow to use uv for linting and testing commands. |
| docs/getting-started.md | Added uv installation instructions alongside existing setup info. |
| examples/team_recommender/readme.md | Updated local setup instructions to use uv commands instead of Poetry. |
| README.md | Revised test and code quality commands and updated documentation links. |
| examples/team_recommender/conftest.py | Removed an unused import to clean up the file. |
Copilot reviewed 9 out of 9 changed files in this pull request and generated no comments.
- Replaced setup from local action to `astral-sh/setup-uv@v5` and `actions/setup-python@v5`. - Updated Python setup to use `python-version-file: "pyproject.toml"`. - Changed project installation command to `uv sync --all-extras --dev`. - Modified test commands to use `uv run` instead of `poetry run`. - Updated documentation link in README Signed-off-by: Tim Kersey <t@k-t.im>
paulz
force-pushed
the
transition-to-uv
branch
from
f0f4f43 to
6275650
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Refactor GitHub workflows to use
uvfor setup and project installationastral-sh/setup-uv@v5andactions/setup-python@v5.python-version-file: "pyproject.toml".uv sync --all-extras --dev.uv runinstead ofpoetry run.