Skip to content

Move bom dependencies to outer pom#2976

Merged
pongad merged 6 commits intogoogleapis:masterfrom
yihanzhen:move-bom-dependencies-to-outer-pom
Mar 1, 2018
Merged

Move bom dependencies to outer pom#2976
pongad merged 6 commits intogoogleapis:masterfrom
yihanzhen:move-bom-dependencies-to-outer-pom

Conversation

@yihanzhen
Copy link
Copy Markdown
Contributor

  • Move guava, protobuf and test dependencies from bom/pom.xml to pom.xml
  • Also fix: bigquery imported truth twice

@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Feb 28, 2018
@yihanzhen
Copy link
Copy Markdown
Contributor Author

PTAL @garrettjonesgoogle @pongad

@pongad
Copy link
Copy Markdown
Contributor

pongad commented Feb 28, 2018

I think we should do point 3 of #2781 (comment) as well, but that can be in a separate PR.

@pongad
Copy link
Copy Markdown
Contributor

pongad commented Feb 28, 2018

@garrettjonesgoogle Do you want to take a look?

Comment thread pom.xml
<dependency>
<groupId>com.google.api.grpc</groupId>
<artifactId>proto-google-cloud-bigtable-v2</artifactId>
<version>${generated-proto-beta.version}</version>

This comment was marked as spam.

@pongad pongad merged commit 597581a into googleapis:master Mar 1, 2018
chingor13 pushed a commit that referenced this pull request Mar 24, 2026
* test: fix ITDatabaseAdminTest failures

* 🦉 Updates from OwlBot post-processor

See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md

* exclude owlbot pre-processor run on kokoro config

* remove reduntant assert

* fix test

---------

Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>
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.

4 participants