Skip to content

Added uv package manager support#42

Merged
ChrisYx511 merged 3 commits intomainfrom
c424yang/migrate-to-uv
Oct 9, 2025
Merged

Added uv package manager support#42
ChrisYx511 merged 3 commits intomainfrom
c424yang/migrate-to-uv

Conversation

@ChrisYx511
Copy link
Contributor

@ChrisYx511 ChrisYx511 commented Oct 1, 2025

For waterloo-rocketry/2025-software-issues#69

  • Added uv.lock and detailed pyproject.toml to replace setup.py
  • Moved to uv_build as the module builder.
  • Moved code into src
  • Updated corresponding documentation

Note: requirements.txt has been kept to maintain compatibility with pip and related utilities.

Build:

  • Run uv sync to pull packages
  • Run uv build to build

This change is Reviewable

@ChrisYx511 ChrisYx511 requested a review from JasonBrave as a code owner October 1, 2025 20:02
@ChrisYx511
Copy link
Contributor Author

ChrisYx511 commented Oct 1, 2025

This change is part of the following stack:

Change managed by git-spice.

@ChrisYx511 ChrisYx511 force-pushed the c424yang/migrate-to-uv branch 3 times, most recently from e1a9fac to 1d6db70 Compare October 1, 2025 20:12
@ChrisYx511 ChrisYx511 force-pushed the c424yang/migrate-to-uv branch from 1d6db70 to dd315f6 Compare October 1, 2025 23:12
@ChrisYx511 ChrisYx511 self-assigned this Oct 3, 2025
vincentjguo
vincentjguo previously approved these changes Oct 6, 2025
Copy link

@vincentjguo vincentjguo left a comment

Choose a reason for hiding this comment

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

Not sure about all those alerts but I think the files you edited are fine

zangjiucheng
zangjiucheng previously approved these changes Oct 7, 2025
Copy link
Member

@zangjiucheng zangjiucheng left a comment

Choose a reason for hiding this comment

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

LGTM, there is known warning about pytest we didn't fix that last year. (Could be fixed later)

Copy link
Member

@JasonBrave JasonBrave left a comment

Choose a reason for hiding this comment

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

LGTM

@ChrisYx511 ChrisYx511 merged commit 0ff7e6e into main Oct 9, 2025
2 of 4 checks passed
@ChrisYx511 ChrisYx511 deleted the c424yang/migrate-to-uv branch October 9, 2025 00:25
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.

4 participants