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

Fix reporting missing required flags #168

Merged
merged 2 commits into from
Jan 7, 2021

Conversation

alexmcmanus
Copy link
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.

index.js Outdated Show resolved Hide resolved
@sindresorhus
Copy link
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

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

chromaui/chromatic-cli#235

@sindresorhus
Copy link
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