Skip to content

Conversation

Mariatta
Copy link
Contributor

@Mariatta Mariatta commented Dec 14, 2021

Description

The examples in other client libraries are all using sampleTask.

Additionally, the doc says:

The following example creates a key with kind Task using a key name, "sampleTask", as the identifier:

Closes internal bug 210583283

Checklist

@Mariatta Mariatta requested review from a team as code owners December 14, 2021 22:49
@product-auto-label product-auto-label bot added the samples Issues that are directly related to samples. label Dec 14, 2021
Copy link
Collaborator

@leahecole leahecole left a comment

Choose a reason for hiding this comment

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

LGTM as long as someone from @GoogleCloudPlatform/cloud-native-db-dpes is happy!

@@ -150,26 +150,26 @@ def incomplete_key(client):

def named_key(client):
# [START datastore_named_key]
key = client.key("Task", "sample_task")
key = client.key("Task", "sampleTask")
Copy link
Contributor

@kolea2 kolea2 Dec 16, 2021

Choose a reason for hiding this comment

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

LGTM! Could we add a test somewhere for these? CC @leahecole as I'm unfamiliar with this repo/structure :)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks. I've added tests to assert that the key.name == "sampleTask".

Mariatta and others added 3 commits January 28, 2022 09:47
The examples in other client libraries are all using `sampleTask`.

Additionally, the doc says: 
> The following example creates a key with kind Task using a key name, "sampleTask", as the identifier:
@leahecole leahecole requested a review from kolea2 January 28, 2022 18:37
@kolea2 kolea2 added the automerge Merge the pull request once unit tests and other checks pass. label Jan 31, 2022
@gcf-merge-on-green gcf-merge-on-green bot merged commit ec3725b into main Jan 31, 2022
@gcf-merge-on-green gcf-merge-on-green bot deleted the Mariatta-patch-1 branch January 31, 2022 19:48
@gcf-merge-on-green gcf-merge-on-green bot removed the automerge Merge the pull request once unit tests and other checks pass. label Jan 31, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
samples Issues that are directly related to samples.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants