-
Notifications
You must be signed in to change notification settings - Fork 6.6k
docs: Change Python client examples from sample_task to sampleTask #7231
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
Conversation
There was a problem hiding this 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") |
There was a problem hiding this comment.
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 :)
There was a problem hiding this comment.
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"
.
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:
b400713
to
aa7b0c0
Compare
Description
The examples in other client libraries are all using
sampleTask
.Additionally, the doc says:
Closes internal bug 210583283
Checklist
nox -s py-3.6
(see Test Environment Setup)nox -s lint
(see Test Environment Setup)