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

Update protoc to v3.20.1 #2731

Merged
merged 2 commits into from
May 5, 2022
Merged

Update protoc to v3.20.1 #2731

merged 2 commits into from
May 5, 2022

Conversation

pav-kv
Copy link
Contributor

@pav-kv pav-kv commented May 4, 2022

Since v3.14.0, protoc started using the new import path for Go types:
google.golang.org/protobuf. This change updates the protobuf compiler
to the latest v3.20.1, to get rid of the deprecated import path and take
advantage of newer versions after v3.14.0.

Checklist

@@ -14,7 +14,6 @@ require (
github.com/go-sql-driver/mysql v1.6.0
github.com/golang/glog v0.0.0-20160126235308-23def4e6c14b
github.com/golang/mock v1.6.0
github.com/golang/protobuf v1.5.2
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Win!

@pav-kv pav-kv changed the title Update protoc to 3.20.1 Update protoc to v3.20.1 May 4, 2022
Since v3.14.0 protoc started using the new import path for Go types:
`google.golang.org/protobuf`. This change updates the protobuf compiler
to the latest v3.20.1, to get rid of the deprecated import path and take
advantage of newer versions after v3.14.0.
@pav-kv pav-kv requested a review from mhutchinson May 4, 2022 20:01
@pav-kv pav-kv marked this pull request as ready for review May 4, 2022 20:01
@pav-kv pav-kv requested a review from a team as a code owner May 4, 2022 20:01
@pav-kv pav-kv merged commit 91360a8 into google:master May 5, 2022
@pav-kv pav-kv deleted the upgrade_protoc branch May 5, 2022 08:46
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.

2 participants