Skip to content

Fix reporting missing required flags#168

Merged
sindresorhus merged 2 commits intosindresorhus:masterfrom
alexmcmanus:165-kebab-flags
Jan 7, 2021
Merged

Fix reporting missing required flags#168
sindresorhus merged 2 commits intosindresorhus:masterfrom
alexmcmanus:165-kebab-flags

Conversation

@alexmcmanus
Copy link
Copy Markdown
Contributor

@alexmcmanus alexmcmanus commented Oct 30, 2020

Fixes #165. Also updated readme.md.

Note that I've made a transitive dependency explicit (decamelize). Rather than add the latest version, I've stuck with the older version brought in by decamelize-keys to avoid unnecessary bloat.

Comment thread index.js Outdated
@sindresorhus
Copy link
Copy Markdown
Owner

You need to update the doc comments in index.d.ts too.

@sindresorhus sindresorhus changed the title #165 missing required flags now reported in kebab-case. Fix reporting missing required flags now in kebab-case Dec 23, 2020
@sindresorhus sindresorhus changed the title Fix reporting missing required flags now in kebab-case Fix reporting missing required flags Jan 7, 2021
@sindresorhus sindresorhus merged commit 1609709 into sindresorhus:master Jan 7, 2021
@MichaelPeter
Copy link
Copy Markdown

FYI: This breaking change just broke chromatic since chromatic was auto upgrade

chromaui/chromatic-cli#235

@sindresorhus
Copy link
Copy Markdown
Owner

#173

Repository owner locked and limited conversation to collaborators Jan 7, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

isRequired does not recognize kebab-case flags

4 participants