Skip to content
This repository was archived by the owner on Mar 16, 2022. It is now read-only.

Merge changes from ELLIOTTCABLE/master#2

Open
AprilArcus wants to merge 3 commits intojsahlen:masterfrom
AprilArcus:ELLIOTTCABLE
Open

Merge changes from ELLIOTTCABLE/master#2
AprilArcus wants to merge 3 commits intojsahlen:masterfrom
AprilArcus:ELLIOTTCABLE

Conversation

@AprilArcus
Copy link
Copy Markdown

On his fork, @ELLIOTTCABLE made these commits:

This PR merges his fork with upstream.

ELLIOTTCABLE and others added 3 commits February 11, 2018 11:43
This plugin previously assumed that `nodenv` used $NODENV_ROOT to store
the directory where *nodenv* was installed. This is not the case. This
fix uses whichever directory `nodenv` is installing Node.js versions in
(preferring the user-local `~/.nodenv/versions` over the system-global
one.)
Critically, previous versions would fuck up the user's `$PATH` if it
happened to include spaces. Unacceptable, yikes.

I also refactored the script slightly, to use consistent POSIX tests,
consistent casing for variable-names, etceteras.
@ELLIOTTCABLE
Copy link
Copy Markdown

Ahhah. Hello. Yes! Enjoy! 🤣

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants