Skip to content

Issue 19#20

Open
JoshWalkerJolt wants to merge 3 commits intomathiasbynens:masterfrom
joltup:issue-19
Open

Issue 19#20
JoshWalkerJolt wants to merge 3 commits intomathiasbynens:masterfrom
joltup:issue-19

Conversation

@JoshWalkerJolt
Copy link
Copy Markdown

Hi. We've had the same problem as the OP from #19
Here's a fix.
All tests passing.
Could you merge this in and release a new version to npm?

);
}
const padding = input.length % 3;
const outputArray = '';
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@JoshWalkerJolt shouldn't it be const outputArray = []?

Copy link
Copy Markdown

@vishalnarkhede vishalnarkhede left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please check my comment!!

@Jorge-Luis-Rangel-Peralta
Copy link
Copy Markdown

I do not think this lib is maintained, is there a way to take control of it without creating a new one?

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.

3 participants