Skip to content

datastore: force import grpc-google-common-protos:0.1.9#2029

Merged
pongad merged 2 commits intogoogleapis:masterfrom
pongad:datastore-versioning
May 4, 2017
Merged

datastore: force import grpc-google-common-protos:0.1.9#2029
pongad merged 2 commits intogoogleapis:masterfrom
pongad:datastore-versioning

Conversation

@pongad
Copy link
Copy Markdown
Contributor

@pongad pongad commented May 3, 2017

Also removes grpc-all dependency from top-level pom.xml; it is unused.

Datastore is tested with mvn verify.
Depedency versions are shown to not conflict by
creating and compiling a test project that depends
only on google-cloud.
Since it does not inherit the version resolution
from google-cloud-java/pom.xml,
it more closely resembles user project.

Also removes grpc-all dependency from top-level pom.xml; it is unused.

Datastore is tested with `mvn verify`.
Depedency versions are shown to not conflict by
creating and compiling a test project that depends
only on google-cloud.
Since it does not inherit the version resolution
from google-cloud-java/pom.xml,
it more closely resembles user project.
@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label May 3, 2017
@coveralls
Copy link
Copy Markdown

Coverage Status

Changes Unknown when pulling 1985918 on pongad:datastore-versioning into ** on GoogleCloudPlatform:master**.

Copy link
Copy Markdown
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.

Please ensure that this works as expected:
rm -r ~/.m2/repository
mvn clean install -DskipITs (from google-cloud-java repo root)

Comment thread google-cloud-datastore/pom.xml Outdated
<dependency>
<groupId>com.google.api.grpc</groupId>
<artifactId>grpc-google-common-protos</artifactId>
<version>0.1.9</version>

This comment was marked as spam.

This comment was marked as spam.

@pongad
Copy link
Copy Markdown
Contributor Author

pongad commented May 3, 2017

@vam-google PTAL

@coveralls
Copy link
Copy Markdown

Coverage Status

Changes Unknown when pulling 6a6dc8a on pongad:datastore-versioning into ** on GoogleCloudPlatform:master**.

Copy link
Copy Markdown
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

<groupId>com.google.api.grpc</groupId>
<artifactId>grpc-google-common-protos</artifactId>
<version>${generatedProto.version}</version>
</dependency>

This comment was marked as spam.

This comment was marked as spam.

This comment was marked as spam.

@pongad
Copy link
Copy Markdown
Contributor Author

pongad commented May 4, 2017

Merging with @vam-google 's +2. @garrettjonesgoogle I'll follow up in email.

@pongad pongad merged commit 5d50249 into googleapis:master May 4, 2017
@pongad pongad deleted the datastore-versioning branch May 4, 2017 04:06
chingor13 pushed a commit that referenced this pull request Feb 20, 2026
suztomo pushed a commit to suztomo/google-cloud-java that referenced this pull request Mar 23, 2026
suztomo pushed a commit to suztomo/google-cloud-java that referenced this pull request Mar 23, 2026
meltsufin pushed a commit that referenced this pull request Apr 29, 2026
meltsufin pushed a commit that referenced this pull request May 1, 2026
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.

5 participants