Skip to content

fix: typo in fs_update_document_increment sample#2427

Merged
andrewferlitsch merged 1 commit intomasterfrom
BenWhitehead-patch-1
Oct 9, 2019
Merged

fix: typo in fs_update_document_increment sample#2427
andrewferlitsch merged 1 commit intomasterfrom
BenWhitehead-patch-1

Conversation

@BenWhitehead
Copy link
Copy Markdown
Contributor

Fixes #2374

@BenWhitehead BenWhitehead requested a review from crwilcox October 1, 2019 18:09
@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Oct 1, 2019
@BenWhitehead
Copy link
Copy Markdown
Contributor Author

Build test hangs are due to googleapis/google-cloud-python#9008 submitted fix is googleapis/google-cloud-python#9337.

@BenWhitehead
Copy link
Copy Markdown
Contributor Author

Waiting for next release of google-cloud-core to include the fix.

@andrewferlitsch
Copy link
Copy Markdown
Contributor

@BenWhitehead Just verifying from your comment, that this PR request is not ready for review until next release of google-cloud-core?

@BenWhitehead
Copy link
Copy Markdown
Contributor Author

@andrewferlitsch Correct. I think a new release is scheduled to be out in the next day or two.

@BenWhitehead
Copy link
Copy Markdown
Contributor Author

@crwilcox the build is green after the latest release. Can you please merge this?

Copy link
Copy Markdown
Contributor

@gguuss gguuss left a comment

Choose a reason for hiding this comment

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

LGTM

@gguuss
Copy link
Copy Markdown
Contributor

gguuss commented Oct 9, 2019

@BenWhitehead Does this fix the listen tests?

@andrewferlitsch andrewferlitsch merged commit 44d1d15 into master Oct 9, 2019
@gguuss
Copy link
Copy Markdown
Contributor

gguuss commented Oct 9, 2019

I just tested, this fixes the listen tests, we should re-enable them.

@BenWhitehead BenWhitehead deleted the BenWhitehead-patch-1 branch October 9, 2019 19:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Mistake in firestore numeric increment example for python

4 participants