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

feat: support useMapType option #686

Merged
merged 1 commit into from
Oct 16, 2022

Conversation

moznion
Copy link
Contributor

@moznion moznion commented Oct 15, 2022

This option controls the way to generate the code for protobuf's map<key_type, value_type>:

  • true: it generates the code with using JavaScript Map type like Map<key_type, value_type>
  • false (default): it generates the code with key-value pair like {[key: key_type]: value_type}

This fixes #547

This option controls the way to generate the code for protobuf's
`map<key_type, value_type>`:

- true: it generates the code with using JavaScript Map type like `Map<key_type, value_type>`
- false (default): it generates the code with key-value pair like `{[key: key_type]: value_type}`

This fixes stephenh#547

Signed-off-by: moznion <[email protected]>
Copy link
Owner

@stephenh stephenh left a comment

Choose a reason for hiding this comment

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

Looks great, thanks @moznion !

@stephenh stephenh merged commit f2e80ab into stephenh:main Oct 16, 2022
stephenh pushed a commit that referenced this pull request Oct 16, 2022
# [1.129.0](v1.128.0...v1.129.0) (2022-10-16)

### Features

* support `useMapType` option ([#686](#686)) ([f2e80ab](f2e80ab))
@stephenh
Copy link
Owner

🎉 This PR is included in version 1.129.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

@moznion moznion deleted the feat_use_map_type_option branch October 16, 2022 20:38
@moznion
Copy link
Contributor Author

moznion commented Oct 16, 2022

I appreciate your work, @stephenh!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Feat: proto map generates a Hashmap instead of an object
2 participants