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.

Sign up for GitHub

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

Jump to bottom

uvx defaults to ruff? #7348

Closed
dimaqq opened this issue Sep 13, 2024 · 3 comments
Closed

uvx defaults to ruff? #7348

dimaqq opened this issue Sep 13, 2024 · 3 comments
Labels
question Asking for clarification or support

Comments

@dimaqq
Copy link

dimaqq commented Sep 13, 2024 

🦐~> uvx
ruff v0.6.3
- ruff

Like, why?

Or, if it's a good idea, pls document it in usage/--help
@zanieb
Copy link
Member

zanieb commented Sep 13, 2024 •
edited
Loading

Huh?

❯ uvx
No tools installed

Ah I see. It's listing your installed tools

❯ uv tool install ruff
Resolved 1 package in 206ms
Prepared 1 package in 787ms
Installed 1 package in 8ms
 + ruff==0.6.4
Installed 1 executable: ruff
❯ uvx
ruff v0.6.4
- ruff
❯ uv tool install black
Resolved 6 packages in 131ms
Prepared 1 package in 85ms
Installed 6 packages in 9ms
 + black==24.8.0
 + click==8.1.7
 + mypy-extensions==1.0.0
 + packaging==24.1
 + pathspec==0.12.1
 + platformdirs==4.3.2
Installed 2 executables: black, blackd
❯ uvx
black v24.8.0
- black
- blackd
ruff v0.6.4
- ruff

@zanieb zanieb added the question Asking for clarification or support label Sep 13, 2024
@dimaqq
Copy link
Author

dimaqq commented Sep 13, 2024

Oh wow... good to know.

I wonder if it's better to fail with usage instead

@charliermarsh
Copy link
Member

I believe this is intended.

kakkoyun added a commit to kakkoyun/uv that referenced this issue Sep 23, 2024
@kakkoyun
Add context message before listing available tools when no arguments …
7471018 
…are provided

Adds a helpful context message when `uvx` is run without arguments
to clarify that it is displaying the installed tools.

This addresses confusion, such as the one highlighted in issue astral-sh#7348,
by making the output more user-friendly and informative.

Signed-off-by: Kemal Akkoyun <kakkoyun@gmail.com>
@kakkoyun kakkoyun mentioned this issue Sep 23, 2024
Merged
kakkoyun added a commit to kakkoyun/uv that referenced this issue Sep 24, 2024
@kakkoyun
Add context message before listing available tools when no arguments …
5cf16d0 
…are provided

Adds a helpful context message when `uvx` is run without arguments
to clarify that it is displaying the installed tools.

This addresses confusion, such as the one highlighted in issue astral-sh#7348,
by making the output more user-friendly and informative.

Signed-off-by: Kemal Akkoyun <kakkoyun@gmail.com>
kakkoyun added a commit to kakkoyun/uv that referenced this issue Sep 26, 2024
@kakkoyun
Add context message before listing available tools when no arguments …
5223ece 
…are provided

Adds a helpful context message when `uvx` is run without arguments
to clarify that it is displaying the installed tools.

This addresses confusion, such as the one highlighted in issue astral-sh#7348,
by making the output more user-friendly and informative.

Signed-off-by: Kemal Akkoyun <kakkoyun@gmail.com>
zanieb pushed a commit that referenced this issue Sep 26, 2024
@kakkoyun
uvx/uv tool run: Add context message before listing available tools w…
3ce3403 
…hen no arguments are provided (#7641)

## Summary

Adds a helpful context message when `uvx` is run without arguments
To clarify, it is displaying the installed tools.

This addresses confusion, such as the one highlighted in issue #7348,
by making the output more user-friendly and informative.

Related #4024 

## Test Plan

Updated the test snapshots to include the new output.
Running the tests locally with `cargo nextest run` confirms that the
tests pass.
The CI pipeline should also pass.

### Manuel Testing

**uvx**
```shell
# Make sure you have the updated version of uv installed on your path.
# cargo install --path ./crates/uv --force
❯ uvx
Provide a command to invoke with `uvx <command>` or `uvx --from <package> <command>`.

The following tools are already installed:

black v24.8.0
- black
- blackd
ruff v0.6.7
- ruff

See `uvx --help` for more information.
```

**uv tool list**
```shell
# Make sure you have the updated version of uv installed on your path.
# cargo install --path ./crates/uv --force
❯ uv tool list
black v24.8.0
- black
- blackd
ruff v0.6.7
- ruff
```

**uv tool run**
```shell
# Make sure you have the updated version of uv installed on your path.
# cargo install --path ./crates/uv --force
❯ uv tool run
Provide a command to invoke with `uv tool run <command>` or `uv tool run --from <package> <command>`.

The following tools are already installed:

black v24.8.0
- black
- blackd
ruff v0.6.7
- ruff

See `uv tool run --help` for more information.
```
---

Signed-off-by: Kemal Akkoyun <kakkoyun@gmail.com>

---------

Signed-off-by: Kemal Akkoyun <kakkoyun@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
question Asking for clarification or support
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@dimaqq @charliermarsh @zanieb