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

Write ints without quotes #109

Merged
merged 5 commits into from Nov 8, 2020
Merged

Write ints without quotes #109

merged 5 commits into from Nov 8, 2020

Conversation

mniak
Copy link
Contributor

@mniak mniak commented Jun 26, 2020

Alternate solution for #107
closes #107

replacement for #108

@kostajh
Copy link

kostajh commented Aug 27, 2020

Seems reasonable to me!

@addshore
Copy link

addshore commented Oct 28, 2020

As #108 is merged, is this still needed?

@joho
Copy link
Owner

joho commented Nov 8, 2020

#108 is closed not merged. Happy to accept this one.

@joho joho merged commit 3e4069b into joho:master Nov 8, 2020
1 check passed
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.

Marshal option that accept non-string values in the map
4 participants