search

sindresorhus / meow

🐈 CLI app helper
MIT License
3.53k stars 150 forks source link

Default `isMultiple` to empty array #163

Closed ulken closed 3 years ago

ulken commented 3 years ago

Note: Changes behavior for booleanDefault: undefined as well. With this change in place, I don't think it makes sense to filter out unset boolean array flags any longer.

Fixes #161

sindresorhus commented 3 years ago

Changes behavior for booleanDefault: undefined as well.

I think the behavior should be made clear in the readme & index.d.ts.

ulken commented 3 years ago

@sindresorhus Gave it a go.

While I was at it, I took the liberty to

  1. Change the structure to adhere to the rest of the README, i.e. break it into two paragraphs. First a sentence to describe what the flag does, followed by a more detailed description of its ins and outs.
  2. Synced README and index.d.ts in terms of content (one line was missing from the README).

If you disapprove, let me know and I'll revert.

sindresorhus commented 3 years ago

Looks good 👍🏻