Skip to content
This repository was archived by the owner on Mar 2, 2026. It is now read-only.

feat!: Begin using new microgenerator for v2 firestore#91

Merged
crwilcox merged 15 commits intogoogleapis:v2-stagingfrom
crwilcox:onboard-microgen
Jul 14, 2020
Merged

feat!: Begin using new microgenerator for v2 firestore#91
crwilcox merged 15 commits intogoogleapis:v2-stagingfrom
crwilcox:onboard-microgen

Conversation

@crwilcox
Copy link
Copy Markdown
Contributor

Commits early on split off to separate out generated bits from edits to (hopefully) ease reviewing

@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Jul 10, 2020
@crwilcox
Copy link
Copy Markdown
Contributor Author

Kokoro Test is Skipping System, which is odd, running locally.

@crwilcox
Copy link
Copy Markdown
Contributor Author

Coverage is at 97%, not 100%. Considering, temporarily, dropping this and filing an issue. Docs issue tbd.

@crwilcox crwilcox mentioned this pull request Jul 13, 2020
@crwilcox crwilcox requested a review from rafilong July 13, 2020 15:40
@crwilcox
Copy link
Copy Markdown
Contributor Author

Comment thread .flake8 Outdated
Comment thread synth.py Outdated
Comment thread synth.py
Comment thread .kokoro/publish-docs.sh
Comment thread synth.py
Copy link
Copy Markdown
Contributor

@rafilong rafilong left a comment

Choose a reason for hiding this comment

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

LGTM! Most of the comments I had were clarifying confusion on my end.

Comment thread google/cloud/firestore_v1/__init__.py Outdated
Comment thread google/cloud/firestore_v1/__init__.py
Comment thread google/cloud/firestore_v1/base_collection.py
Comment thread google/cloud/firestore_v1/base_document.py
Comment thread google/cloud/firestore_v1/base_query.py
Comment thread tests/unit/v1/test_collection.py Outdated
Comment thread tests/unit/v1/test_document.py Outdated
Comment thread tests/unit/v1/test_transaction.py Outdated
Comment thread tests/unit/v1/test_watch.py
Comment thread tests/unit/v1/test_watch.py
Comment thread google/cloud/firestore_v1/order.py Outdated
Comment thread google/cloud/firestore_v1/order.py
Copy link
Copy Markdown
Contributor

@rafilong rafilong left a comment

Choose a reason for hiding this comment

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

lgtm!

@crwilcox crwilcox merged commit e0add08 into googleapis:v2-staging Jul 14, 2020
@release-please release-please bot mentioned this pull request Aug 20, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants