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: declare typing for 'firestore_v1' and 'firestore_bundle' packages #507

Closed
wants to merge 9 commits into from

Conversation

tseaver
Copy link
Contributor

@tseaver tseaver commented Dec 22, 2021

Test them, along with 'tests/' in the 'mypy' nox session.

Closes #447.

Supersedes #495.

@tseaver tseaver requested a review from a team as a code owner December 22, 2021 22:06
@product-auto-label product-auto-label bot added the api: firestore Issues related to the googleapis/python-firestore API. label Dec 22, 2021
@tseaver tseaver requested a review from tswast December 23, 2021 20:00
@meredithslota meredithslota removed their assignment May 13, 2022
@meredithslota meredithslota requested review from Mariatta and removed request for tswast May 13, 2022 22:59
@Mariatta Mariatta added the type: cleanup An internal cleanup or hygiene concern. label Dec 1, 2022
@meredithslota meredithslota removed the request for review from Mariatta May 16, 2023 20:59
@daniel-sanche
Copy link
Contributor

I'm going to close this as outdated - too many conflicts.

I am starting to look at fixing the mypy typing as part of #773

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api: firestore Issues related to the googleapis/python-firestore API. type: cleanup An internal cleanup or hygiene concern.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

google.cloud.firestore lacks packaged typing
5 participants