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.

Sign up for GitHub

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

Jump to bottom

adding support for experimental_allow_proto3_optional #78

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

adding support for experimental_allow_proto3_optional #78

wants to merge 1 commit into from

Conversation

melocotton
Copy link

cf. #77

@melocotton melocotton mentioned this pull request Oct 8, 2020
Open
@clehene
Copy link

clehene commented Mar 21, 2021

@sergei-ivanov this would be useful as it enables field presence
An option to allow arbitrary flags would be even better

@vntzy
Copy link

vntzy commented Mar 31, 2021

Hello! Is this planned to get merged and released any time soon? Thanks.

@melocotton
Copy link
Author

Only the owner of this repository is allowed to merge this PR ?

@omardelarosa
Copy link

Has there been any movement on this PR? I'd also love to see this get merged, or broader support for feature flags like this one since this maven plugin is quite useful but seems to be incompatible with newer features of protobufs.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@omardelarosa omardelarosa omardelarosa approved these changes

@vntzy vntzy vntzy approved these changes

Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@melocotton @clehene @vntzy @omardelarosa @sebhvrsg