Skip to content

Update uninstall command's help text with examples of what can actually be uninstalled#1628

Merged
chriskrycho merged 4 commits intovolta-cli:mainfrom
MrChocolatine:patch-1
Jan 1, 2024
Merged

Update uninstall command's help text with examples of what can actually be uninstalled#1628
chriskrycho merged 4 commits intovolta-cli:mainfrom
MrChocolatine:patch-1

Conversation

@MrChocolatine
Copy link
Copy Markdown
Contributor

@MrChocolatine MrChocolatine commented Dec 26, 2023

Following this comment from @chriskrycho:
#1563 (comment)

Resolves #1563

@MrChocolatine MrChocolatine changed the title Remove node and npm from uninstall command's help text Update uninstall command's help text with examples of what can actually be uninstalled Dec 26, 2023
Copy link
Copy Markdown
Contributor

@charlespierce charlespierce left a comment

Choose a reason for hiding this comment

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

LGTM! Thanks for taking this on!

At the time of writing this, uninstalling this tool is not supported:

```
➜ volta uninstall node
error: Uninstalling node is not supported yet.
```

Resolves volta-cli#1563
At the time of writing this, uninstalling this tool is not supported:

```
➜ volta uninstall npm
error: Uninstalling npm is not supported yet.
```

Resolves volta-cli#1563
At the time of writing this, uninstalling this tool is not supported:

```
➜ volta uninstall yarn
error: Uninstalling yarn is not supported yet.
```
With examples of what can actually be uninstalled.
@MrChocolatine
Copy link
Copy Markdown
Contributor Author

MrChocolatine commented Dec 31, 2023

Rebased and green again @charlespierce .

@chriskrycho chriskrycho merged commit fc8ff66 into volta-cli:main Jan 1, 2024
@chriskrycho
Copy link
Copy Markdown
Contributor

Thanks again!

@MrChocolatine MrChocolatine deleted the patch-1 branch January 1, 2024 17:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Help text for volta uninstall --help incorrectly states node versions can be uninstalled

3 participants