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.

Sign up for GitHub

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

Jump to bottom

Add MustStringArray to match StringArray() #40

Merged
merged 1 commit into from
Mar 31, 2015
Merged

Add MustStringArray to match StringArray() #40

merged 1 commit into from
Mar 31, 2015

Conversation

mattheath
Copy link
Contributor

There are already StringArray() and MustArray() functions, but no MustStringArray(). This adds it in a consistent way with the current interface.

@mreiferson
Copy link
Contributor

thanks @mattheath !!!

mreiferson added a commit that referenced this pull request Mar 31, 2015
@mreiferson
Merge pull request #40 from mattheath/stringarray
18db6e6 
Add MustStringArray to match StringArray()
@mreiferson mreiferson merged commit 18db6e6 into bitly:master Mar 31, 2015
@mreiferson
Copy link
Contributor

@mattheath try https://github.com/mreiferson/go-ujson - we have an opportunity to clean up this API there (and it's faster decoding anyway)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@mattheath @mreiferson