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

add long options #6

Merged
merged 2 commits into from
Jun 6, 2021
Merged

add long options #6

merged 2 commits into from
Jun 6, 2021

Conversation

marchersimon
Copy link

I added long options for all the flags, since I think they are easier to remember. BTW, I'm also documenting the page in tldr-pages/tldr#6098 right now, you could leave a comment if you want to.

@ym1234 ym1234 merged commit bfb0fce into ym1234:master Jun 6, 2021
@ym1234
Copy link
Owner

ym1234 commented Jun 6, 2021

Thanks

@marchersimon marchersimon deleted the long-options branch June 6, 2021 12:57
@marchersimon
Copy link
Author

marchersimon commented Jun 6, 2021

No problem ;)
However there's a mistake @bl-ue pointed out which got merged. (The argument is --oneshot and I wrote --one-shot in the help part). Should I make a new PR for this or do you want to change it yourself?

@bl-ue
Copy link

bl-ue commented Jun 6, 2021

#7 :)

@ym1234
Copy link
Owner

ym1234 commented Jun 6, 2021

I like --one-shot more + oneshot isn't a word anyway.

@bl-ue
Copy link

bl-ue commented Jun 6, 2021

@ym1234 I've updated that PR. It's mergeable now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants