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 JSON-escaped characters in string array #1177

Merged
merged 1 commit into from
Dec 10, 2016

Conversation

charmander
Copy link
Collaborator

Improved fix for #1147 over #1148.

@brianc
Copy link
Owner

brianc commented Dec 10, 2016

@charmander - you're amazing. Would you like collaborator access to the repo so in any case where I'm out I don't end up blocking you? I'd love to have more co-maintainers helping out!

@brianc
Copy link
Owner

brianc commented Dec 10, 2016

Since CI is borked (you fixed it!! but on another PR) I'll run the tests locally & if they pass I'll get this one merged in.

@brianc
Copy link
Owner

brianc commented Dec 10, 2016

Works! Merged! Thank you! ❤️

@brianc brianc merged commit 5d821c3 into brianc:master Dec 10, 2016
@charmander
Copy link
Collaborator Author

@brianc Sure, thanks! And thank you for going through all this again.

@charmander charmander deleted the fix-newlines-in-string-array branch December 11, 2016 00:33
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