Update maven config to include *.proto files in jars by BenWhitehead · Pull Request #5208 · googleapis/google-cloud-java · GitHub
Skip to content

Update maven config to include *.proto files in jars#5208

Merged
chingor13 merged 1 commit into
googleapis:masterfrom
BenWhitehead:include-proto-files-in-jars
May 22, 2019
Merged

Update maven config to include *.proto files in jars#5208
chingor13 merged 1 commit into
googleapis:masterfrom
BenWhitehead:include-proto-files-in-jars

Conversation

@BenWhitehead

Copy link
Copy Markdown
Contributor

Add build-helper-maven-plugin to com.google.api.grpc:google-api-grpc
thereby updating the config for all proto-* modules such that when
their jars are built they will now include the *.proto. This allows
dependents to already have the proto files on their classpath when they
are defining/generating their own protos.

Add build-helper-maven-plugin config to
com.google.cloud:google-cloud-firestore for its test resources as
proto is defined for the conformance tests.

Add build-helper-maven-plugin to `com.google.api.grpc:google-api-grpc`
thereby updating the config for all `proto-*` modules such that when
their jars are built they will now include the `*.proto`. This allows
dependents to already have the proto files on their classpath when they
are defining/generating their own protos.

Add build-helper-maven-plugin config to
`com.google.cloud:google-cloud-firestore` for its test resources as
proto is defined for the conformance tests.
@BenWhitehead BenWhitehead requested a review from a team May 21, 2019 21:31
@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label May 21, 2019
@codecov

codecov Bot commented May 21, 2019

Copy link
Copy Markdown

@sduskis sduskis requested review from chingor13 and removed request for a team May 22, 2019 18:26
@chingor13 chingor13 merged commit ef1f4f2 into googleapis:master May 22, 2019
@BenWhitehead BenWhitehead deleted the include-proto-files-in-jars branch May 22, 2019 22:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants