Skip to content

allow support for falsy values like 0 in tagged literal#45

Merged
bcoe merged 2 commits intoyargs:masterfrom
JamesKyburz:falsy-support-tagged-literal
Aug 10, 2017
Merged

allow support for falsy values like 0 in tagged literal#45
bcoe merged 2 commits intoyargs:masterfrom
JamesKyburz:falsy-support-tagged-literal

Conversation

@JamesKyburz
Copy link
Copy Markdown
Contributor

Without the undefined check the lookup key with %s won't be found when passing falsy values.

@bcoe
Copy link
Copy Markdown
Member

bcoe commented May 17, 2017

@JamesKyburz good catch 👍 mind adding a little regression test when you get two secs?

@JamesKyburz
Copy link
Copy Markdown
Contributor Author

will do!

@JamesKyburz
Copy link
Copy Markdown
Contributor Author

Sorry for the delay, I have now added a test for falsy arguments to the pull request :)

@JamesKyburz
Copy link
Copy Markdown
Contributor Author

Could this please be merged?

@bcoe
Copy link
Copy Markdown
Member

bcoe commented Aug 10, 2017

@JamesKyburz sorry for the slow reply 👍 have been trying to dig myself out of an OSS backlog.

Thanks for the contribution!

@bcoe bcoe merged commit c926123 into yargs:master Aug 10, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants