Should escape grep in nvm_ls #468

Merged
merged 1 commit into from Jul 12, 2014

Projects

None yet

2 participants

@egilkh

Without the \grep escaping I kept getting output like this:

$ nvm ls
   (standard
input):2:v0.10.29

and

nvm use 0.10
(standard input):1:v0.10.29 version is not installed yet

Because I have grep to a crazy alias.

Strange thing is that tests worked in both cases... But this fixed the problem of nvm ls and nvm use for me.

@ljharb
Collaborator

The test operates in its own shell which wouldn't necessarily have your alias. Thanks for this fix.

@ljharb ljharb merged commit c49cb68 into creationix:master Jul 12, 2014

1 check passed

Details continuous-integration/travis-ci The Travis CI build passed
@ljharb ljharb added the bugs label Jul 12, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment