Skip to content

proto: add protoc-gen-govalidators import#2532

Merged
jayconrod merged 1 commit intobazel-contrib:masterfrom
johanbrandhorst:add-mwitkow-govalidators-dep
Jun 5, 2020
Merged

proto: add protoc-gen-govalidators import#2532
jayconrod merged 1 commit intobazel-contrib:masterfrom
johanbrandhorst:add-mwitkow-govalidators-dep

Conversation

@johanbrandhorst
Copy link
Copy Markdown
Contributor

@johanbrandhorst johanbrandhorst commented Jun 5, 2020

Fixes #2531

See https://github.com/mwitkow/go-proto-validators/blob/master/examples/enum.validator.pb.go#L11 for an example of a generated file that imports this package. I'm using the validators_golang since that is the stack we're supporting with the existing plugin. See https://github.com/mwitkow/go-proto-validators/blob/master/BUILD.bazel#L83

This import is part of the generated files and was missing.

Fixes bazel-contrib#2531
Copy link
Copy Markdown
Collaborator

@jayconrod jayconrod left a comment

Choose a reason for hiding this comment

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

Looks good, thanks!

@jayconrod jayconrod merged commit 3a62a42 into bazel-contrib:master Jun 5, 2020
@johanbrandhorst johanbrandhorst deleted the add-mwitkow-govalidators-dep branch June 8, 2020 08:39
jayconrod pushed a commit that referenced this pull request Jul 8, 2020
This import is part of the generated files and was missing.

Fixes #2531
yushan26 pushed a commit to yushan26/rules_go that referenced this pull request Jun 16, 2025
…-contrib#2532)

The warning is somewhat non-actionable and the sources can be
inspected via the MODULE.bazel.lock file if needed. This makes it
easier to make this option a default at some point.

At the same time cleanup the code since we are not using the
`get_index_urls` to print the warning.

Work towards bazel-contrib#260
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.

go_proto_validate is missing a dependency

3 participants