Skip to content

Conversation

@bepri
Copy link
Member

@bepri bepri commented Jan 21, 2025

  • Have you signed the CLA?

Adding the uv plugin from craft-parts to Rockcraft.

CRAFT-3916
Closes #789.

@bepri bepri self-assigned this Jan 21, 2025
@bepri bepri force-pushed the work/uv-plugin/CRAFT-3916 branch from c3022c8 to ff0e5fa Compare January 24, 2025 22:32
@bepri
Copy link
Member Author

bepri commented Jan 28, 2025

Test failures should all be unrelated:

  • python plugin test likely flaky after a craft-parts update
  • poetry plugin tests failing after poetry export was removed from default poetry installs
  • OSV errors appear to come from unpinned tutorial dependencies

@bepri bepri marked this pull request as ready for review January 28, 2025 15:50
@bepri bepri requested review from lengau and tigarmo January 28, 2025 15:50
Copy link
Collaborator

@tigarmo tigarmo left a comment

Choose a reason for hiding this comment

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

The docs failure is related:


checking consistency... /home/docs/checkouts/readthedocs.org/user_builds/canonical-rockcraft/checkouts/792/docs/common/craft-parts/reference/plugins/go_use_plugin.rst: WARNING: document isn't included in any toctree
/home/docs/checkouts/readthedocs.org/user_builds/canonical-rockcraft/checkouts/792/docs/common/craft-parts/reference/plugins/uv_plugin.rst: WARNING: document isn't included in any toctree

@tigarmo
Copy link
Collaborator

tigarmo commented Jan 28, 2025

@lengau do you know what's up with this OSV-scanner? I don't know which requirements file it's picking up here:
image

@bepri
Copy link
Member Author

bepri commented Jan 28, 2025

I think the failure is coming from this being unpinned: https://github.com/canonical/rockcraft/blob/main/docs/tutorial/code/fastapi/requirements.txt

@tigarmo tigarmo added the squash label Jan 28, 2025
Copy link
Collaborator

@tigarmo tigarmo left a comment

Choose a reason for hiding this comment

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

great!

@medubelko medubelko self-requested a review January 28, 2025 21:05
Co-authored-by: Alex Lowe <alex.lowe@canonical.com>
@tigarmo tigarmo merged commit 449dad9 into main Jan 29, 2025
15 of 18 checks passed
@tigarmo tigarmo deleted the work/uv-plugin/CRAFT-3916 branch January 29, 2025 15:33
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.

Add support for the uv plugin

5 participants