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

chore: add py.typed file for PEP 561 compatibility#241

Merged
parthea merged 2 commits intomainfrom
add-pytyped
Oct 13, 2021
Merged

chore: add py.typed file for PEP 561 compatibility#241
parthea merged 2 commits intomainfrom
add-pytyped

Conversation

@crwilcox
Copy link
Contributor

Fixes #240

@crwilcox crwilcox requested a review from a team as a code owner October 13, 2021 00:37
@google-cla google-cla bot added the cla: yes This human has signed the Contributor License Agreement. label Oct 13, 2021
@product-auto-label product-auto-label bot added the api: datastore Issues related to the googleapis/python-datastore API. label Oct 13, 2021
@parthea parthea merged commit dbe20e3 into main Oct 13, 2021
@parthea parthea deleted the add-pytyped branch October 13, 2021 08:32
@tswast
Copy link
Contributor

tswast commented Oct 13, 2021

Similar to Bigtable, I don't see any checks to verify that mypy and pytype are passing. This likely will break folks that actually do use those tools if our experience in BigQuery applies.

@crwilcox
Copy link
Contributor Author

Do note, you may need to run mypy, pyright, pytype against our samples before merging. It will ensure that we aren't exposing users to failing type checks.

Ideally we would add this to CI as well, to avoid future breakage.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

api: datastore Issues related to the googleapis/python-datastore API. cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add a py.typed file for submodule

3 participants