Skip to content
This repository has been archived by the owner on Jul 20, 2023. It is now read-only.

test: deploy mode test failure #145

Merged
merged 4 commits into from
Jun 4, 2021
Merged

test: deploy mode test failure #145

merged 4 commits into from
Jun 4, 2021

Conversation

telpirion
Copy link
Contributor

This PR attempts to address the following issues: #74 and #75 .

@product-auto-label product-auto-label bot added the api: aiplatform Issues related to the AI Platform API. label Jun 2, 2021
@google-cla google-cla bot added the cla: yes This human has signed the Contributor License Agreement. label Jun 2, 2021
Copy link
Contributor

@JustinBeckwith JustinBeckwith left a comment

Choose a reason for hiding this comment

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

The fix prefix indicates to our release tooling that this is a change to the library itself, which should trigger a release. I think test: would be more appropriate here.

@@ -18,8 +18,8 @@
},
"devDependencies": {
"chai": "^4.2.0",
"gts": "^3.0.2",
Copy link
Contributor

Choose a reason for hiding this comment

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

So generally we don't include gts in the samples folder, and rely on it being installed at the top level package.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done. Good to know!

@telpirion telpirion changed the title fix: deploy mode test failure test: deploy mode test failure Jun 3, 2021
@telpirion telpirion marked this pull request as ready for review June 3, 2021 18:20
@telpirion telpirion requested review from a team as code owners June 3, 2021 18:20
@telpirion telpirion requested a review from JustinBeckwith June 3, 2021 18:20
@JustinBeckwith JustinBeckwith merged commit 0e1e1ba into master Jun 4, 2021
@JustinBeckwith JustinBeckwith deleted the deploy-model-fix branch June 4, 2021 03:14
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
api: aiplatform Issues related to the AI Platform API. cla: yes This human has signed the Contributor License Agreement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants