Skip to content

Conversation

@glerchundi
Copy link
Contributor

@glerchundi glerchundi commented Feb 21, 2021

By bumping the version we gain the ability to add the supported features
to the code generation response which helps by not erroring out when it
gets called with the newly released protobuf 3.15. As the experimental
flag for optionals was made mandatory if an optional clause exist in
the source proto files.

Signed-off-by: Gorka Lerchundi Osa glertxundi@gmail.com

@glerchundi
Copy link
Contributor Author

glerchundi commented Feb 21, 2021

xref. protoc-gen-star PR: lyft/protoc-gen-star#85

@dkunitsk
Copy link
Contributor

Hey @glerchundi. Let me see if I can approve your PGS PR. @rodaine has moved on from Lyft, and that repo has been in a bit of limbo, but I think I can help. Let me look into it before you try to merge this.

@glerchundi
Copy link
Contributor Author

Thanks @dkunitsk!!

@rauanmayemir
Copy link

Is there any update on this?
We want to update protobuf, but currently stuck with older version.

@glerchundi
Copy link
Contributor Author

As far as I know nothing has changed since I opened the issues

@glerchundi glerchundi changed the title go.mod: replace lyft's with glerchundi's protoc-gen-star go.mod: bump protoc-gen-star to v0.5.3 May 4, 2021
@glerchundi
Copy link
Contributor Author

Scope of the PR changed as the changes in protoc-gen-star were merged and a new version released.

@glerchundi glerchundi mentioned this pull request May 4, 2021
2 tasks
By bumping the version we gain the ability to add the supported features
to the code generation response which helps by not erroring out when it
gets called with the newly released protobuf 3.15. As the experimental
flag for optionals was made mandatory if an `optional` clause exist in
the source proto files.

Signed-off-by: Gorka Lerchundi Osa <glertxundi@gmail.com>
@akonradi akonradi merged commit dd3f943 into bufbuild:main May 4, 2021
@glerchundi
Copy link
Contributor Author

TFYR @akonradi!! 🎉

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.

4 participants