Skip to content
This repository was archived by the owner on Sep 19, 2023. It is now read-only.

Conversation

@stephaniewang526
Copy link
Contributor

Fixes #177

@stephaniewang526 stephaniewang526 requested a review from a team as a code owner October 16, 2020 19:08
@google-cla google-cla bot added the cla: yes This human has signed the Contributor License Agreement. label Oct 16, 2020
Comment on lines 52 to 78
# java-bigquery
git clone https://github.com/googleapis/java-bigquery.git
cd java-bigquery
mvn install
cd google-cloud-bigquery
mvn validate -D${VERSION_KEY}=${VERSION}

# java-bigqueryconnection
git clone https://github.com/googleapis/java-bigqueryconnection.git
cd java-bigqueryconnection
mvn install
cd google-cloud-bigqueryconnection
mvn validate -D${VERSION_KEY}=${VERSION}

# java-storage
git clone https://github.com/googleapis/java-storage.git
cd java-storage
mvn install
cd google-cloud-storage
mvn validate -D${VERSION_KEY}=${VERSION}

# java-spanner
git clone https://github.com/googleapis/java-spanner.git
cd java-spanner
mvn install
cd google-cloud-spanner
mvn validate -D${VERSION_KEY}=${VERSION} No newline at end of file
Copy link
Contributor

@chingor13 chingor13 Oct 16, 2020

Choose a reason for hiding this comment

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

Let's parameterize this script and run it in parallel (on actions?).

cd java-bigqueryconnection
mvn install
cd google-cloud-bigqueryconnection
mvn validate -D${VERSION_KEY}=${VERSION}
Copy link
Contributor

Choose a reason for hiding this comment

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

Do we need to run all the tests? or just run the dependencies tests?

Co-authored-by: Jeff Ching <chingor@google.com>
@stephaniewang526
Copy link
Contributor Author

implemented in #183

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.

Add CI test to verify there is no upperbound errors amongst dependencies managed through this BOM downstream

2 participants