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

Support to encode map which has defined type key #231

Merged
merged 3 commits into from
Jul 2, 2021

Conversation

kitagry
Copy link
Contributor

@kitagry kitagry commented Jun 26, 2021

This PR aims to fix #230 .

@codecov-commenter
Copy link

codecov-commenter commented Jun 26, 2021

Codecov Report

Merging #231 (036fc04) into master (2bc7bb1) will increase coverage by 0.01%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #231      +/-   ##
==========================================
+ Coverage   76.80%   76.81%   +0.01%     
==========================================
  Files          12       12              
  Lines        2957     2959       +2     
==========================================
+ Hits         2271     2273       +2     
  Misses        461      461              
  Partials      225      225              

Copy link
Owner

@goccy goccy left a comment

Choose a reason for hiding this comment

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

Thank you for the contribution ! I commented only one place .

encode.go Outdated Show resolved Hide resolved
@kitagry
Copy link
Contributor Author

kitagry commented Jun 30, 2021

@goccy
Thank you for the review! I fixed.

@goccy
Copy link
Owner

goccy commented Jul 2, 2021

LGTM ! Thank you for the great fixing !!!

@goccy goccy merged commit f5a7c47 into goccy:master Jul 2, 2021
@kitagry kitagry deleted the support-defined-type-key branch July 2, 2021 08:16
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.

Cannot marshal map which key is defined type
3 participants