Skip to content
This repository has been archived by the owner on Dec 6, 2023. It is now read-only.

Fix bug when trying to select already selected version. #1

Merged
merged 2 commits into from
Oct 7, 2014

Conversation

gcirne
Copy link
Contributor

@gcirne gcirne commented Oct 7, 2014

Steps to reproduce bug (without this patch):

1 - nvm use 0.10.32
2 - .node-version with 0.10.32 (same as already selected version)
3 - cd to directory containing above .node-version
4 - error returned: avn could not activate node 0.10.32

Steps to reproduce bug (without this patch):

1 - nvm use 0.10.32
2 - .node-version with 0.10.32 (same as already selected version)
3 - cd to directory containing above .node-version
4 - error returned: avn could not activate node 0.10.32
@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling 084445c on gcirne:fix-bug-already-selected-version into 475f047 on wbyoung:master.

@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling cbf42a4 on gcirne:fix-bug-already-selected-version into 475f047 on wbyoung:master.

@wbyoung wbyoung merged commit cbf42a4 into wbyoung:master Oct 7, 2014
@wbyoung
Copy link
Owner

wbyoung commented Oct 7, 2014

@gcirne thanks. I made a few more changes that should make it compatible with current and older versions of nvm and added some tests.

A new version's been published. Let me know if you have any issues with it.

@gcirne
Copy link
Contributor Author

gcirne commented Oct 8, 2014

Great @wbyoung! Just tested the new version here and everything is working correctly.

Thanks!

@gcirne gcirne deleted the fix-bug-already-selected-version branch October 8, 2014 17:05
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.

3 participants