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

fix(protoc-gen-swagger): Windows compatibility #1070

Merged

Conversation

mrmeku
Copy link
Contributor

@mrmeku mrmeku commented Oct 28, 2019

Inspired by protobuf.bzl from the main bazel repo:
https://github.com/bazelbuild/bazel/blob/deeeba3fc8003cbbcc73e04f14fc5b5275e72e5d/third_party/protobuf/3.6.1/protobuf.bzl#L108

This PR introducing a windows mapping from protoc-gen-swagger to the binary's actual path.

Inspired by protobuf.bzl from the main bazel repo:
https://github.com/bazelbuild/bazel/blob/deeeba3fc8003cbbcc73e04f14fc5b5275e72e5d/third_party/protobuf/3.6.1/protobuf.bzl#L108

This PR introducing a windows mapping from protoc-gen-swagger to the
binaries actual path.
@codecov-io
Copy link

Codecov Report

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

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #1070   +/-   ##
=======================================
  Coverage   53.85%   53.85%           
=======================================
  Files          42       42           
  Lines        4176     4176           
=======================================
  Hits         2249     2249           
  Misses       1681     1681           
  Partials      246      246

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 c3787b4...3ff661b. Read the comment docs.

@johanbrandhorst johanbrandhorst merged commit 55de1dd into grpc-ecosystem:master Oct 28, 2019
@johanbrandhorst
Copy link
Collaborator

LGTM

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.

4 participants