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

chore(protoc): Bump protoc and protoc-gen-go #71

Merged
merged 1 commit into from
May 27, 2020
Merged

chore(protoc): Bump protoc and protoc-gen-go #71

merged 1 commit into from
May 27, 2020

Conversation

ezimanyi
Copy link
Contributor

This gets us to 3.12.x of protoc, which adds support for optional fields. We said we'd revisit in a few weeks (a few weeks ago) and make a final decision about how to handle this. By pulling in a version of protoc that has support for optional fields (behind a flag) it will be easier to play around with this and see the generated code than if we also had to bump protoc.

Also bump protoc-gen-go while we're generating this code.

As usual, neither dependency bump changes the generated code, other than the versioned comment.

This gets us to 3.12.x of protoc, which adds support for optional
fields. We said we'd revisit in a few weeks (a few weeks ago) and
make a final decision about how to handle this. By pulling in a
version of protoc that has support for optional fields (behind a
flag) it will be easier to play around with this and see the
generated code than if we also had to bump protoc.

Also bump protoc-gen-go while we're generating this code.

As usual, neither dependency bump changes the generated code,
other than the versioned comment.
@ezimanyi ezimanyi requested a review from maggieneterval May 27, 2020 21:07
Copy link
Contributor

@maggieneterval maggieneterval left a comment

Choose a reason for hiding this comment

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

🔢 ⬆️ 🎉 🙏

@ezimanyi ezimanyi added the ready to merge Reviewed and ready for merge label May 27, 2020
@mergify mergify bot merged commit 9d60ed2 into spinnaker:master May 27, 2020
@mergify mergify bot added the auto merged label May 27, 2020
@ezimanyi ezimanyi deleted the bump-protoc branch May 27, 2020 21:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
auto merged ready to merge Reviewed and ready for merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants