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

@perenecabuto perenecabuto commented Oct 29, 2018

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

@geoah
Copy link

@geoah 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

@xcaptain xcaptain commented Dec 19, 2018

Need this feature, hope to get merged.

@wjh000123
Copy link

@wjh000123 wjh000123 commented Feb 22, 2019

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

@estebangarcia
Copy link

@estebangarcia estebangarcia commented Jun 7, 2019

It would be great if this gets merged!

@cweerasooriya
Copy link

@cweerasooriya cweerasooriya commented Jul 26, 2019

Great feature. 💯
Is this getting merged anytime soon?

Copy link

@felipemfp felipemfp left a comment

LGTM

@hsyed
Copy link

@hsyed hsyed commented Oct 18, 2019

could this be merged please ?

@tdakkota
Copy link

@tdakkota tdakkota commented Nov 19, 2019

Why haven't merged it yet?

@ideruga
Copy link

@ideruga ideruga commented Nov 20, 2019

Is it possible to merge this?

@kmanco
Copy link

@kmanco kmanco commented Nov 20, 2019

Can you merge this?

@soluchok
Copy link

@soluchok soluchok commented Jan 8, 2020

@mitchellh Is this repo maintainable by someone?

@Dragomir-Ivanov
Copy link

@Dragomir-Ivanov Dragomir-Ivanov commented Jan 20, 2020

@mitchellh this merge appreciated!

@crlssn
Copy link

@crlssn crlssn commented Mar 23, 2020

I would like to see this merged as well!

@mitchellh mitchellh merged commit ca9bdad into mitchellh:master Apr 28, 2020
1 check passed
1 check passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet