Add python type hinting #758
Labels
api: datastore
Issues related to the googleapis/python-ndb API.
type: feature request
‘Nice-to-have’ improvement, new feature or different behavior or design.
This is work that might be contributed by folks other than the maintainers.
Is your feature request related to a problem? Please describe.
Checking types in python has been loose and folks sometimes get themselves in trouble when trying to modify older code bases. It's not always obvious to the reader of the code what type(s) should be in play for particular code. Having type hints in python-ndb would help a bit with that.
Describe the solution you'd like
If the project is up for it, say that you are okay with PRs that setup testing of type hinting (pytype and/or mypy) and then adding type hinting. Then maybe we can round up some contributors.
Describe alternatives you've considered
It's okay to not add type hinting. It solves only a particular set of issues. And users of the library should have test coverage exercising all of their use, but that can sometimes be a difficult bar to achieve.
Additional context
I personally have found I'm faster at modifying code with type hints and create fewer bugs in that code. YMMV.
Additionally there needs to be a couple decisions:
from __future__ import annotations
?Any
? e.g.Dict[str, Any]
for JSON rather than something super messyThe text was updated successfully, but these errors were encountered: