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

Should support -v argument to check version #78

Closed
meetwudi opened this issue Feb 7, 2014 · 10 comments
Closed

Should support -v argument to check version #78

meetwudi opened this issue Feb 7, 2014 · 10 comments
Labels
clients Issues pertaining to a particular client or the clients as whole.

Comments

@meetwudi
Copy link
Contributor

meetwudi commented Feb 7, 2014

I am happy to see tldr now supports multi platforms. But when I run tldr zip, not found error was given. I used npm update to update tldr, but it is still not working. Maybe the new version is not published yet? But I am not able to check the version.

@rprieto
Copy link
Contributor

rprieto commented Feb 7, 2014

Very good point thanks.

Version 0.0.5 should bring multi-platform support, but someone reported an issue downloading it from npm. I'll investigate to see why that is.

Can you see what version you current have in /usr/local/lib/node_modules/tldr ?

@meetwudi
Copy link
Contributor Author

meetwudi commented Feb 7, 2014

0.0.3 @rprieto

@rprieto
Copy link
Contributor

rprieto commented Feb 7, 2014

That's definitely not the latest. What happens when you npm install -g tldr again?

@meetwudi
Copy link
Contributor Author

meetwudi commented Feb 7, 2014

Well, it got updated if I run npm install -g tldr. :)

@rprieto
Copy link
Contributor

rprieto commented Feb 7, 2014

That's great. I'm not sure npm update updates global dependencies. Let's keep this open until we add a -v flag though.

@svogt
Copy link

svogt commented Feb 7, 2014

I think you can use npm update -g

@meetwudi
Copy link
Contributor Author

meetwudi commented Feb 7, 2014

@svogt It did not work :(

@meetwudi
Copy link
Contributor Author

meetwudi commented Feb 7, 2014

@rprieto I suggest commander.js which simplifies the workflow of processing process.argv and generate help messages when user runs tldr -h. It is pretty.

@rprieto
Copy link
Contributor

rprieto commented Feb 8, 2014

@tjwudi thanks, commander is definitely nice to use. Someone already pulled in optimist to do the first arg parsing, so I might stick with that to get a quick patch for -v in.

However substack announced he was "retiring" optimist, so in the future commander, yargs or nomnom could be a good long term solution.

@rprieto rprieto closed this as completed in 492e53e Feb 9, 2014
@rprieto
Copy link
Contributor

rprieto commented Feb 9, 2014

Just published version 0.0.6, with support for tldr -V or tldr --version.
I ended up switching to commander anyway :)

@CleanMachine1 CleanMachine1 added the clients Issues pertaining to a particular client or the clients as whole. label Jul 11, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
clients Issues pertaining to a particular client or the clients as whole.
Projects
None yet
Development

No branches or pull requests

4 participants