search

sindresorhus / meow

🐈 CLI app helper
MIT License
3.54k stars 151 forks source link

Only consider enabling autoHelp/autoVersion in case there is only one argument in `process.argv` #114

Closed LitoMore closed 5 years ago

LitoMore commented 5 years ago

Resolved #113

sindresorhus commented 5 years ago

The behavior needs to be documented.

LitoMore commented 5 years ago

@sindresorhus Code updated. I documented this behavior in the Features section, is that OK?

LitoMore commented 5 years ago

For the document, I renamed the Tips section to Related, and add a new section named Tips for autoVersion and autoHelp tips.

LitoMore commented 5 years ago

@sindresorhus ping

LitoMore commented 5 years ago

Any progress? @sindresorhus

sindresorhus commented 5 years ago

add a new section named Tips for autoVersion and autoHelp tips.

This should be document under the docs for those options, not in a Tips section. It's not a tip, it's documenting how something works.

LitoMore commented 5 years ago

@sindresorhus Docs updated

sindresorhus commented 5 years ago

The PR title needs to be updated to reflect the final behavior.

LitoMore commented 5 years ago

@sindresorhus Updated

sindresorhus commented 5 years ago

Thanks :)