Skip to content
This repository has been archived by the owner on Jun 28, 2022. It is now read-only.

Add sample_config.proto to the config_java_proto BUILD target #2852

Merged
merged 2 commits into from
Jun 24, 2019

Conversation

vchudnov-g
Copy link
Contributor

We accidentally omitted this in #2828

@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Jun 22, 2019
@vchudnov-g
Copy link
Contributor Author

fyi @noahdietz @hzyi-google @alexander-fenster

Copy link
Contributor

@vam-google vam-google left a comment

Choose a reason for hiding this comment

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

LGTM (with minor comment)

BUILD.bazel Outdated
java_proto_library(
name = "config_java_proto",
deps = [":config_proto",
":config_v2_proto"],
":config_v2_proto",
":sample_config_proto"],
Copy link
Contributor

Choose a reason for hiding this comment

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

Looks like this mixes tabs and spaces. Please convert it to spaces-only.

@vchudnov-g vchudnov-g merged commit 1b1ee16 into googleapis:master Jun 24, 2019
alexander-fenster added a commit to googleapis/artman that referenced this pull request Jun 26, 2019
Changes included are fixes:

- samplegen: add sample_config.proto to the config_java_proto BUILD target googleapis/gapic-generator#2852
- go: URL encode x-goog-request-params googleapis/gapic-generator#2851
- ruby: ake it possible to pass metadata to the operations client googleapis/gapic-generator#2856
alexander-fenster added a commit to googleapis/artman that referenced this pull request Jun 26, 2019
Changes included are fixes:

- samplegen: add sample_config.proto to the config_java_proto BUILD target googleapis/gapic-generator#2852
- go: URL encode x-goog-request-params googleapis/gapic-generator#2851
- ruby: ake it possible to pass metadata to the operations client googleapis/gapic-generator#2856
alexander-fenster added a commit to googleapis/artman that referenced this pull request Jun 26, 2019
Changes included in this release are fixes:

In artman:

- pass the specific YAML loader to yaml.load #648 
- fix --local runs #650

In gapic-generator:

- samplegen: add sample_config.proto to the config_java_proto BUILD target googleapis/gapic-generator#2852
- go: URL encode x-goog-request-params googleapis/gapic-generator#2851
- ruby: ake it possible to pass metadata to the operations client googleapis/gapic-generator#2856
alexander-fenster added a commit to googleapis/artman that referenced this pull request Jun 26, 2019
Changes included in this release are fixes:

In artman:

- pass the specific YAML loader to yaml.load #648 
- fix --local runs #650

In gapic-generator:

- samplegen: add sample_config.proto to the config_java_proto BUILD target googleapis/gapic-generator#2852
- go: URL encode x-goog-request-params googleapis/gapic-generator#2851
- ruby: ake it possible to pass metadata to the operations client googleapis/gapic-generator#2856
busunkim96 pushed a commit to busunkim96/gapic-generator that referenced this pull request Nov 7, 2019
…apis#2852)

* Add sample_config.proto to the config_java_proto BUILD target

* Cosmetic: convert tab to space
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
cla: yes This human has signed the Contributor License Agreement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants