-
Notifications
You must be signed in to change notification settings - Fork 413
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
Implement uv tree
#4708
base: main
Are you sure you want to change the base?
Implement uv tree
#4708
Conversation
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.
Looks reasonable to me. Nice.
30eb67f
to
579bc1e
Compare
crates/uv/tests/common/mod.rs
Outdated
@@ -302,7 +302,7 @@ impl TestContext { | |||
if cfg!(all(windows, debug_assertions)) { | |||
// TODO(konstin): Reduce stack usage in debug mode enough that the tests pass with the | |||
// default windows stack of 1MB | |||
command.env("UV_STACK_SIZE", (8 * 1024 * 1024).to_string()); |
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.
Please update the contributing guide too
Fix windows stack sizes for #4708 In this case, box clap arguments so that clap parsing can be done in the default 1MB stack on windows.
The changes in #4708 caused an overflow in debug mode only of the 1MB default stack size in windows during clap. This means that even trivial wrong argument tests would fail without increasing the stack size. As remedy, we box the clap types.
Fix windows stack sizes for #4708 In this case, box clap arguments so that clap parsing can be done in the default 1MB stack on windows.
The changes in #4708 caused an overflow in debug mode only of the 1MB default stack size in windows during clap. This means that even trivial wrong argument tests would fail without increasing the stack size. As remedy, we box the clap types.
ddcad05
to
4d7dca4
Compare
4d7dca4
to
ad38914
Compare
I expect that #4796 (box the project commands type) will also fix the tests here. How do you want to proceeed @ibraheemdev? I can make the changes from #4796 but it will cause merge conflicts with both this and the |
Summary
Implements the
uv tree
, which displays dependencies from the lockfile as a tree. Resolves #4699.