Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Support using more recent datastore emulator versions via gcloud emulators #1303

Merged
merged 1 commit into from
Oct 5, 2016

Conversation

mziccard
Copy link
Contributor

@mziccard mziccard commented Oct 3, 2016

This fixes #1300 for good

@mziccard mziccard added the api: datastore Issues related to the Datastore API. label Oct 3, 2016
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.04%) to 84.672% when pulling a6f5b9c on mziccard:gcd-version into b61af33 on GoogleCloudPlatform:master.

@mziccard
Copy link
Contributor Author

mziccard commented Oct 4, 2016

/cc @lesv

@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Oct 4, 2016
@lesv
Copy link
Contributor

lesv commented Oct 4, 2016

LGTM - if only folks always kept versions like you expect them. It's likely the emulator will keep the correct scheme, but lots of teams use 0.1.2-1.9.42, 0.2.3-beta, 0.2.3-SNAPSHOT, or 0.2.3-rev172 etc.

@mziccard
Copy link
Contributor Author

mziccard commented Oct 4, 2016

@lesv I was strict on the version format on purpose. I prefer the helper to download its own release version of the emulator rather than possibly using a beta/alpha version already installed via gcloud. I agree this should not happen with emulators.

@mziccard mziccard merged commit 46f808b into googleapis:master Oct 5, 2016
github-actions bot pushed a commit that referenced this pull request Aug 9, 2022
Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api: datastore Issues related to the Datastore API. cla: yes This human has signed the Contributor License Agreement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

It is too slow testing Datastore locally when use LocalDatastoreHelper
4 participants