victorb / trymodule

➰ It's never been easier to try nodejs modules!
1.14k stars 29 forks source link

Added a "clear" command #4

Closed jacksarick closed 8 years ago

jacksarick commented 8 years ago

Added a "clear" command. trymodule -clear removes $HOME/.trymodules/node_modules

victorb commented 8 years ago

Not sure this is much better than simply doing rm -r ~/.trymodule but I'll leave this here for a bit and will think about it.

Thanks for the contribution :)

victorb commented 8 years ago

Ok, I'm happy to merge this but there are some changes that has to be done.

This may seem like a lot but it's tiny changes, some stylistic and some logical ones. It seems you would have to pull the latest changes before continuing as well.

Let me know if you need any help with any of the items :)

victorb commented 8 years ago

@jacksarick something that can help in development (after you updated your branch with git pull origin master [if origin is this repository]), is running npm run lint to figure out if you're following the style standards or not.

victorb commented 8 years ago

I went ahead and implemented this in commit 69c56d71772f352a505f7ecbf9da567cdb2e847d