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

Add explicit dependency versions #696

Merged
merged 2 commits into from
Jul 23, 2018

Conversation

johanbrandhorst
Copy link
Collaborator

Version constraints are copied from existing Bazel rules. In the future, these version upgrades must be performed atomically.

Fixes #689

@johanbrandhorst
Copy link
Collaborator Author

The build errors imply files have been generated with protoc-gen-go 1.1, but Bazel dependencies are using golang/protobuf v1.0.0: #695 (comment).

I think we need to update the rules_go version we're using to something more recent, and at the same time update the locked dependency versions in Gopkg.toml.

This has gotten out of sync with generated files. See
https://github.com/bazelbuild/rules_go/releases/tag/0.12.1
for release notes.
@johanbrandhorst johanbrandhorst force-pushed the add-deps branch 2 times, most recently from 2681c93 to 357d1d4 Compare July 8, 2018 18:11
Version constraints are copied from existing Bazel rules. In the
future, these version upgrades must be performed atomically.
@codecov-io
Copy link

codecov-io commented Jul 8, 2018

Codecov Report

Merging #696 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #696   +/-   ##
=======================================
  Coverage   56.54%   56.54%           
=======================================
  Files          30       30           
  Lines        3040     3040           
=======================================
  Hits         1719     1719           
  Misses       1152     1152           
  Partials      169      169

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 5948c50...7d1b84f. Read the comment docs.

@johanbrandhorst
Copy link
Collaborator Author

Bump. I think we need to get this in. @achew22 @tmc @yugui

@achew22 achew22 merged commit 905e714 into grpc-ecosystem:master Jul 23, 2018
dmacthedestroyer pushed a commit to dmacthedestroyer/grpc-gateway that referenced this pull request Aug 2, 2018
* Add explicit dependency versions (grpc-ecosystem#696)

Version constraints are copied from existing Bazel rules. In the
future, these version upgrades must be performed atomically.

* Add OpenTracing support to docs (grpc-ecosystem#705)

* protoc-gen-swagger: support all well-known wrapper types

There were a few well-known wrapper types missing from
the wkSchemas map. In specific UInt32Value, UInt64Value
and BytesValue. This change handles them (and maps them to
the same swagger types as the non-wrapped values)

This also fixes the mapping of Int64Value. The Int64Value handling
maps the value to a swagger integer. The documentation for
Int64Value indicates that it should be mapped to a JSON
string (also the mapping for normal int64 in protoc-gen-swagger
maps it to a string, so this was inconsistent.)

* Add test case and proposed fix for path component with trailing colon (and string) (grpc-ecosystem#708)

* If a pattern has no verb, match with a verb arg appended to last component

Parsing out a "verb" from the input path is a convenience for
parsing. Whether the colon and string are _actually_ a verb depends on
the matching HTTP rule. This change gives a verb-less pattern a chance
to match a path by assuming the colon+string suffix are a part of the
final component.

Fixes grpc-ecosystem#224

Signed-off-by: James Hamlin <james@goforward.com>

* Fix up examples

* Make support paths option

Make protoc-gen-grpc-gateway support paths option such like golang/protobuf#515.
adasari pushed a commit to adasari/grpc-gateway that referenced this pull request Apr 9, 2020
Version constraints are copied from existing Bazel rules. In the
future, these version upgrades must be performed atomically.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants