New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

numStrings=0 returns non-empty array #16

Closed

fasiha opened this issue Sep 21, 2021 · 0 comments

Owner

fasiha commented Sep 21, 2021

var m=require('mudder');
m.base62.mudder('p', '', 0) // [ 'u' ] 😭

Because I, ignoramus, used a foolish default: numStrings = numStrings || 1;

The text was updated successfully, but these errors were encountered:

fasiha added a commit that referenced this issue


          test to show #16

f106470

fasiha added a commit that referenced this issue


          fix #16

a92a765

fasiha closed this as completed in

4bc243b

# for free to join this conversation on GitHub. Already have an account? # to comment