Returning Unknown instead of Any for incomplete __getattr__

[Avasam]

I noticed we have quite a few

def __getattr__(name: str) -> Incomplete: ...

Wouldn't it be better to write as

def __getattr__(name: str): ...  # incomplete

So that type-checkers like Pyright in strict mode can warn about unknown types ?

[srittau]

I think that was a pattern mandated in CONTRIBUTING for incomplete stubs. I think the current pattern just happened via the route

1. -> Any: ... # incomplete
2. -> Incomplete: ... # incomplete
3. -> Incomplete: ...

That said, I don't have a strong preference either way.