Skip to content

Commit

Permalink
Install OR update
Browse files Browse the repository at this point in the history
  • Loading branch information
@fresheneesz
fresheneesz committed Oct 5, 2015
1 parent 06a2128 commit 49f2c95
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion README.markdown
View file
Expand Up @@ -13,7 +13,7 @@ Note: `nvm` does not support [Fish] either (see [#303](https://github.com/creati

### Install script

This comment has been minimized.

Sign in to view

Copy link
@mockturtl

mockturtl Oct 12, 2018

Consider "Install / Update"?

This comment has been minimized.

Sign in to view

Copy link
@ljharb

ljharb Oct 12, 2018

Member

I suppose; but I still believe that every piece of software ever, unless separately documented, updates by "installing in place" - so this sort of callout shouldn't be needed.

This comment has been minimized.

Sign in to view

Copy link
@mockturtl

mockturtl Oct 13, 2018

I still believe that every piece of software ever, unless separately documented, updates by "installing in place"

It arguably should. Sadly, not every installer is responsible about side-effects (or dependencies, documentation, UX).

If an update breaks: do I purge and reinstall? (sometimes yes)
If I have an old version, can I upgrade directly, or must I step through intermediate versions? (sometimes yes)
Etc.

As a new user, I would rather follow instructions than guess. (That's why I found #400)

As a maintainer, I think added clarity is a good trade-off for a small redundancy in a README.

Anyway, thank you for your work! nvm got me out of a jam with an electron app on Debian oldstable. 🍻

To install you could use the [install script][2] using cURL:
To install or update nvm, you can use the [install script][2] using cURL:

curl -o- https://raw.githubusercontent.com/creationix/nvm/v0.28.0/install.sh | bash

Expand Down

0 comments on commit 49f2c95

Please sign in to comment.