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.

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

Automatically detect latest release #1770

Closed
wants to merge 1 commit into from
Closed

Automatically detect latest release #1770

wants to merge 1 commit into from

Conversation

Xaqron
Copy link

@Xaqron Xaqron commented Apr 2, 2018

Due to constant change of version number it is bothering to update README file. Provided solution resolves this problem.

Copy link
Member

@ljharb ljharb left a comment

Choose a reason for hiding this comment

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

This is already automated by nvm's release process, so it's not actually bothersome.

I think having a version number in the command is helpful, to know what you're installing - and I think that curling to github's API is a bit of a brittle and convoluted approach (but I can't think of a better one)

@ljharb ljharb added the installing nvm Problems installing nvm itself label Apr 2, 2018
@Xaqron
Copy link
Author

Xaqron commented Apr 2, 2018

Better approach would be having the latest script at a constant address as openvpn or docker do.
Every time I gonna install node via NVM I should check what is the latest release so I guessed maybe this is a common problem to other people.

@ljharb
Copy link
Member

ljharb commented Apr 2, 2018

That's by design; you should know what version of software you're installing and using, otherwise how can you know what features it has and what the API is?

@Xaqron
Copy link
Author

Xaqron commented Apr 2, 2018

Exactly. The problem is with design according to best practices. Why big companies like openvpn and docker provide a static address for the latest stable release?
Currently any script trying to install latest NVM should be updated manually or by some code like I have provided in PR. Example of static URLs:

@ljharb
Copy link
Member

ljharb commented Apr 2, 2018

Big companies do a lot of terrible things :-) the fact that a big company does something in no way automatically makes it a good practice.

I have http://latest.nvm.sh which redirects to https://github.com/creationix/nvm/releases/latest, but that's so you can get the manual install command you need from there, as a human.

@Xaqron
Copy link
Author

Xaqron commented Apr 2, 2018

I appreciate your work. Just tried to make life easier :)
Q.E.D

@ljharb ljharb closed this Sep 29, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
installing nvm Problems installing nvm itself
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants