argh icon indicating copy to clipboard operation
argh copied to clipboard

Add -h as a short alias for --help

Open aminya opened this issue 3 years ago • 2 comments

Fixes #30

This adds -h as a short alias for --help

aminya avatar Sep 01 '21 05:09 aminya

Thanks for your pull request. It looks like this may be your first contribution to a Google open source project (if not, look below for help). Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

:memo: Please visit https://cla.developers.google.com/ to sign.

Once you've signed (or fixed any issues), please reply here with @googlebot I signed it! and we'll verify it.


What to do if you already signed the CLA

Individual signers
Corporate signers

ℹ️ Googlers: Go here for more info.

google-cla[bot] avatar Sep 01 '21 05:09 google-cla[bot]

@googlebot I signed it!

aminya avatar Sep 01 '21 05:09 aminya

@sadmac7000 I submitted this pull request in 2021. After these years, is it worth making this simple change configurable? If I was the maintainer, I would have merged it and bumped up the major version.

aminya avatar Oct 19 '23 23:10 aminya

argh exists specifically to implement the Fuchsia command line argument guidelines. Breaking behaviors outside of those guidelines need more than just a major revision bump. They need an update to the guidelines or to be disabled by default.

sadmac7000 avatar Oct 23 '23 01:10 sadmac7000

@sadmac7000 Where is the Fuschia documentation so that I can make the same pull request for that?

aminya avatar Nov 02 '23 22:11 aminya

The guidelines are documented in Google's git hosting here: https://fuchsia.googlesource.com/fuchsia/+/refs/heads/main/docs/development/api/cli.md

sadmac7000 avatar Nov 06 '23 23:11 sadmac7000

@sadmac7000 Reimplemented the help trigger logic to make it configurable through attributes.

aminya avatar Nov 12 '23 14:11 aminya

Looks good. Looks like there's a merge conflict. Mind rebasing?

sadmac7000 avatar Nov 12 '23 17:11 sadmac7000

@sadmac7000 You're welcome. The branch is already rebased. I noticed the GitHub UI can be outdated, so maybe refresh your page.

aminya avatar Nov 12 '23 21:11 aminya

It's still claiming the branch cannot be rebased due to conflicts and won't let me merge. I refreshed a few times. I'll look again later in case it's a transient error.

sadmac7000 avatar Nov 13 '23 17:11 sadmac7000

@sadmac7000 I gave it a shock. Check if it resolved the issue. Haha

aminya avatar Nov 13 '23 20:11 aminya

I had to pull it down and manually rearrange it. In the future, please rebase your changes when you need to update rather than merging in upstream. It confuses github.

sadmac7000 avatar Nov 13 '23 21:11 sadmac7000