-
Notifications
You must be signed in to change notification settings - Fork 621
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
Add support for --with-editable
to uv tool ru
#6744
Conversation
crates/uv/tests/tool_run.rs
Outdated
// If invalid, we should reference `--with`. | ||
uv_snapshot!(context.filters(), context.tool_run().arg("--with").arg("./foo").arg("flask").arg("--version"), @r###" | ||
success: false | ||
exit_code: 2 | ||
----- stdout ----- | ||
|
||
----- stderr ----- | ||
error: Distribution not found at: file://[TEMP_DIR]/foo | ||
"###); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just a quick comment, looks like this doesn't reference --with
like we'd hope :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for your review :)
I'll try working on it for a few more days 👍🏽
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've implemented the feature similarly to the pull request, but I'm running into an issue where an anyhow::Error
is being returned, which is preventing me from adding context to the error. I need to investigate this further, but if you have any insights or know of a possible cause, I would greatly appreciate your input.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@zanieb
Could you please check it once?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
73085ae has the relevant fix
b6a3cd2
to
cca6718
Compare
Sorry for the delay! It's been a busy few weeks. I'll get to this soon. |
--with-editable
to uv tool
--with-editable
to uv tool
--with-editable
to uv tool ru
Summary
close #6272
Test Plan
As in #6262