-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Show tool entry points in uv tool list
#4654
Comments
The proposal to match |
Hello I would like to have a look and help on this; but I am relatively new to uv and rust so it will not be immediate, if this is urgent I will just collaborate and have a look at the proposed solution to learn about the codebase and be useful someday else :) |
Thanks @moreaupascal56! Appreciate it :) It's likely someone will tackle this soon (or I will) but feel free to ask questions on the pull request. We'll be tagging more things as
good first issue
|
I like it, another possible design could be $ uv tool list
black
blackd (from black) By that, no indentions applied to output and user will know what names are available to use.
with this design it'd be:
with indentions:
first one prioritizes entrypoints visually, and more looks good into my eyes. |
I kind of like that, but it gets a little tricky with versions. Nit: If you've used |
<!-- Thank you for contributing to uv! To help us out with reviewing, please consider the following: - Does this pull request include a summary of the change? (See below.) - Does this pull request include a descriptive title? - Does this pull request include references to any relevant issues? --> Closes #4654 ## Summary The purpose of this is to show the entrypoints of each tool when running `uv tool list` as below: ``` $ uv tool list black black blackd ``` I used the proposed formatting as it was written in #4653 by @blueraft. I had to use spaces instead of tabs in order to make the test successful. Indeed in the test we are using a raw string and I did not manage to make the test pass when escaping the tab in the list.rs file so I used spaces everywhere. I had a deeper look into #4653 as well but it is more difficult as we need to get the version of the tool in the Tool object, I will continue on this next one later. Please tell me if anything else is needed I tried to follow the contribution guidelines but I might have forgotten something. Have a great day! ## Test Plan `cargo clippy` then by using the local version of uv as described in the Readme.md. ``` my-computer :~/mypath/uv$ cargo run -- tool list Compiling uv-cli v0.0.1 (/mypath/uv/crates/uv-cli) Compiling uv v0.2.18 (/mypath/uv/crates/uv) Finished `dev` profile [unoptimized + debuginfo] target(s) in 18.69s Running `target/debug/uv tool list` warning: `uv tool list` is experimental and may change without warning. black black blackd isort isort isort-identify-imports ``` and `cargo test tool_list` --------- Co-authored-by: Zanie Blue <contact@zanie.dev>
We should display the name of the executable entry points that are provided by an installed tool e.g.
The format is up for debate, open questions include things like:
Another idea is like
When proposing a design here, we should take into account the output of other tools like
pipx
.Related:
uv tool list
#4653The text was updated successfully, but these errors were encountered: