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

go.mod: update grpc-gateway to v2 #9

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

Conversation

JeffSwenson
Copy link

@JeffSwenson JeffSwenson commented Aug 28, 2023

Attempting to build grpc-gateway v1 using the latest version of io_bazel_go_rules
fails due to a breaking change in how io_bazel_go_rules resolves googleapi
protobufs (https://github.com/bazelbuild/rules_go/releases/tag/v0.41.0).

Version v2.15.2 was chosen because it is a relatively recent version
that occurs before the breaking change was fixed in grpc-gateway. This
allows cockroachdb to import the change without bumping the io_bazel_rules_go
version. Other repositories that want to bump their io_bazel_rules_go can
explicitly depend on a version of grpc-gatewy that includes the bazel fix.

@JeffSwenson JeffSwenson force-pushed the jeffswenson-update-grpc-gateway branch 3 times, most recently from 0727715 to bd1986b Compare August 28, 2023 18:25
Attempting to build grpc-gateway v1 using the latest version of
io_bazel_go_rules fails due to a breaking change in how
io_bazel_go_rules resolves googleapi protobufs
(https://github.com/bazelbuild/rules_go/releases/tag/v0.41.0).

Version v2.15.2 was chosen because it is a relatively recent version
that occurs before the breaking change was fixed in grpc-gateway. This
allows cockroachdb to import the change without bumping the
io_bazel_rules_go version. Other repositories that want to bump their
io_bazel_rules_go can explicitly depend on a version of grpc-gatewy that
includes the bazel fix.
@JeffSwenson JeffSwenson force-pushed the jeffswenson-update-grpc-gateway branch from bd1986b to 04c2181 Compare August 28, 2023 18:26
@JeffSwenson JeffSwenson changed the title go.mod: update grpc-gateway to the latest v2 go.mod: update grpc-gateway to v2 Aug 28, 2023
Jeremyyang920 added a commit that referenced this pull request Feb 13, 2024
This commit takes the changes from #9
and applies them as a patch on top of new snapshots until we
resolve the dependency issues.
Jeremyyang920 added a commit that referenced this pull request Feb 13, 2024
This commit takes the changes from #9
and applies them as a patch on top of new snapshots until we
resolve the dependency issues.
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.

None yet

1 participant