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

Add support for omitempty in JSON tags #3117

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

abdullah2993
Copy link

Adds a new configuration option json_tags_omit_empty which adds omitempty to JSON tags.

Closes #1098
Closes #1084

Related To:
#1132
#1087

Adds a new configuration option json_tags_omit_empty
@flexagoon
Copy link

What's the progress on this? Is this PR just waiting for a review?

@gregoryjjb
Copy link

Thoughts on an option for setting omitempty on nullable fields only? Slightly more of a pain to implement since JSONTagName doesn't know the nullability of its field.

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