Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

CI: Use uv package and project manager on GHA and RTD #569

Merged
merged 2 commits into from
Feb 7, 2025
Merged

CI: Use uv package and project manager on GHA and RTD #569

merged 2 commits into from
Feb 7, 2025

Conversation

amotl
Copy link
Member

@amotl amotl commented Jan 24, 2025

About

Add the uv program to the toolbox, in this case to speed up documentation rebuilds, both on GHA and RTD.

Details

uv is very versatile, and provides better support to work with virtualenvs in different situations, unique support for one-off scripts with inline dependencies, and much more.

References

@amotl amotl changed the title CI: Use uv on RTD CI: Use uv package and project manager Jan 24, 2025
@amotl amotl changed the title CI: Use uv package and project manager CI: Use uv package and project manager on GHA and RTD Jan 24, 2025
@amotl amotl requested review from kneth and surister January 24, 2025 21:51
@amotl amotl marked this pull request as ready for review January 24, 2025 21:51
Copy link

@surister surister left a comment

Choose a reason for hiding this comment

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

👌

Comment on lines 1 to 7
# Install the local crate-docs-theme package by specifying parent directory,
# because `pip` is run from there.
--editable=.
# Install the `crate-docs-theme` package from the local working tree
# in editable mode. `pip` is invoked from the root directory.
#
# When using `uv`, don't use `--editable=.`.
# https://github.com/astral-sh/uv/issues/10941

-e .
Copy link
Member Author

Choose a reason for hiding this comment

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

This has been fixed/improved swiftly by @charliermarsh, thank you!

Let's wait for a new release of uv, then revert this change again. It does not make sense to merge it this way with a misleading indication that something would not be right. Given the current high release cadence, it will probably not be far away.

Copy link
Member Author

Choose a reason for hiding this comment

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

Using --editable=. works well now, thank you Charlie!

@amotl amotl merged commit 8b995d3 into main Feb 7, 2025
8 checks passed
@amotl amotl deleted the uv branch February 7, 2025 23:44
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.

3 participants