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 omitempty #145

Merged
merged 2 commits into from Apr 28, 2020
Merged

Support to omitempty #145

merged 2 commits into from Apr 28, 2020

Conversation

perenecabuto
Copy link
Contributor

Add support to omitempty anotation in order to not add map keys with empty values

@geoah
Copy link

geoah commented Nov 10, 2018

@perenecabuto thanks for this - works wonderfully!
@mitchellh are they any plans on getting this merged or any reasons not to?

@xcaptain
Copy link

Need this feature, hope to get merged.

@wjh000123
Copy link

@mitchellh Hi, are there any plan on getting this merged?

@estebangarcia
Copy link

It would be great if this gets merged!

@cweerasooriya
Copy link

Great feature. 💯
Is this getting merged anytime soon?

Copy link

@felipemfp felipemfp left a comment

Choose a reason for hiding this comment

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

LGTM

@hsyed
Copy link

hsyed commented Oct 18, 2019

could this be merged please ?

@tdakkota
Copy link

Why haven't merged it yet?

@ideruga
Copy link

ideruga commented Nov 20, 2019

Is it possible to merge this?

@ghost
Copy link

ghost commented Nov 20, 2019

Can you merge this?

@soluchok
Copy link

soluchok commented Jan 8, 2020

@mitchellh Is this repo maintainable by someone?

@Dragomir-Ivanov
Copy link

@mitchellh this merge appreciated!

@crlssn
Copy link

crlssn commented Mar 23, 2020

I would like to see this merged as well!

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.

None yet