-
Notifications
You must be signed in to change notification settings - Fork 173
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
gh-754: Allow undefined count #791
Conversation
205fd60
to
f2c9545
Compare
@Shinigami92 does it make sense to you? |
Yeah, could make sense. |
fb3f14b
to
047de67
Compare
bef0c45
to
e726740
Compare
e726740
to
2d3c40a
Compare
hi @Shinigami92! Thanks for the tip to use default value 👍 i think it's ready to merge. Would appreciate if you take a look and approve |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Closes gh-754
Allow count be
undefined
... looks like it was supported before in js version as well as cli doesn't provide it but then count is typed as compulsory.TODO: