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

feat(firestore): add v1beta1 deprecation annotation #34

Merged
merged 7 commits into from Jun 25, 2020

Conversation

mf2199
Copy link
Contributor

@mf2199 mf2199 commented Apr 20, 2020

A 'v1beta1' deprecation annotation added to __init__.py, following the scheme from Bigtable and a related, already merged deprecation PR #8886.

A note on Kokoro failure: The error traces to a docstring inside /google/cloud/firestore_v1/types.py file, which was unaffected by the proposed changes.

@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Apr 20, 2020
@mf2199 mf2199 closed this Apr 20, 2020
@mf2199 mf2199 reopened this Apr 22, 2020
@mf2199 mf2199 changed the title Added v1beta1 deprecation annotation feat(firestore): add v1beta1 deprecation annotation Apr 22, 2020
@IlyaFaer IlyaFaer added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label May 26, 2020
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label May 26, 2020
@IlyaFaer IlyaFaer requested review from BenWhitehead and removed request for lukesneeringer May 26, 2020 08:58
@BenWhitehead
Copy link
Collaborator

@crwilcox Can you take a look at this and review it for the correct way to annotating the v1beta1 protos as deprecated?

@BenWhitehead BenWhitehead removed their request for review May 28, 2020 17:55
@crwilcox crwilcox merged commit b9e2ab5 into googleapis:master Jun 25, 2020
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.

None yet

6 participants