CLN, TYP Remove string type hints

[MarcoGorelli]

• Ensure all linting tests pass, see here for how to run them

For some background: there was some consensus on doing this here

Then here there was the suggestion to use ast.parse, rather than regular expressions. I had a go at that, and made this into its own package: https://github.com/MarcoGorelli/no-string-hints , which I've included here as a pre-commit hook

The changes here are just a result of running pre-commit run no-string-hints -a, and then adding from __future__ import annotations to some files.

[jreback]

wow. lgtm. cc @simonjayhawkins

just have to make sure the author sticks around :->

[rhshadrach]

This is great! Just one question below.

[rhshadrach]

Curious what happened here. Series is imported inline below, so am I correct to say that this is sufficient for mypy?

[MarcoGorelli]

Thanks!

And yes, seems like it, e.g.:
t.py:

def foo() -> None:
    a: List[int]
    reveal_type(a)
    from typing import List

$ python3.8 -m mypy t.py 
t.py:3: note: Revealed type is 'builtins.list[builtins.int]'

[simonjayhawkins]

Thanks @MarcoGorelli. this is great.

[MarcoGorelli]

Thanks all for your kind words, much appreciated 🙏 !