Skip to content

feat(cli): Add uv to cli#537

Merged
pedro-avalos merged 12 commits intomainfrom
add-uv-to-cli
Apr 16, 2025
Merged

feat(cli): Add uv to cli#537
pedro-avalos merged 12 commits intomainfrom
add-uv-to-cli

Conversation

@pedro-avalos
Copy link
Collaborator

@pedro-avalos pedro-avalos commented Mar 31, 2025 •
edited
Loading

Description

  • Add .python-version to specify the Python version we are deploying with
  • Adds uv as the package manager for the CLI package
  • Use tox-uv for running tox tests with uv.
  • Use uv plugin for snapcraft build

Resolved issues

Documentation

Added HACKING.md with docs on how to use uv and develop Testflinger CLI.

Web service API changes

Tests

  • Tox tests should pass with new changes.
  • Built the package with snapcraft locally and tested

@pedro-avalos pedro-avalos marked this pull request as ready for review March 31, 2025 14:32
@pedro-avalos pedro-avalos requested a review from boukeas March 31, 2025 14:32
@pedro-avalos pedro-avalos changed the title chore(cli): Add uv to cli feat(cli): Add uv to cli Mar 31, 2025
boukeas
boukeas reviewed Apr 1, 2025
View reviewed changes
boukeas
boukeas reviewed Apr 1, 2025
View reviewed changes
pedro-avalos
pedro-avalos commented Apr 1, 2025
View reviewed changes
@pedro-avalos pedro-avalos requested a review from boukeas April 2, 2025 20:06
@pedro-avalos pedro-avalos mentioned this pull request Apr 3, 2025
Merged
5 tasks
@pedro-avalos pedro-avalos force-pushed the add-uv-to-cli branch 2 times, most recently from e12899b to 8ec90bd Compare April 8, 2025 19:26
@pedro-avalos
Run tests with different python versions
223951d 
Also remove unnecessary `py` environment from env list
@pedro-avalos
Move set_env in tox.ini
9b123e5
@pedro-avalos
Add tox test to ensure lock file is up to date
6066249 
This is useful in case someone adds/removes a dependency manually to pyproject.toml.
boukeas
boukeas reviewed Apr 16, 2025
View reviewed changes
@pedro-avalos
Remove extraneous testing environments
2d8ba39 
we are only deploying to 3.10
boukeas
boukeas reviewed Apr 16, 2025
View reviewed changes
boukeas
boukeas approved these changes Apr 16, 2025
View reviewed changes
Copy link
Contributor

@boukeas boukeas left a comment

Choose a reason for hiding this comment

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

Thanks Pedro and apologies for taking too long to review 🙂

@pedro-avalos pedro-avalos merged commit bddefb9 into main Apr 16, 2025
2 checks passed
@pedro-avalos pedro-avalos deleted the add-uv-to-cli branch April 16, 2025 14:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@boukeas boukeas boukeas approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@boukeas boukeas

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@pedro-avalos @boukeas