Add custom return type annotation for subscriptions about strawberry

[DoctorJohn]

Do we still want this @patrick91? Personally I'm fine with AsyncGenerator[str, None] since I see it a lot in code using asyncio. But adding a type alias is still an option of course :)

(sorry for digging this up, I'm trying to get our issue count under control :D )

[patrick91]

I think an alias would be nice to have, at least until we get optional type params :D

[patrick91]

@jaydenwindle yes, that's a very good point, I don't like that too, not sure what our best options are.

The reason why I was using typing.AsyncGenerator is because MyPy was complaining :D

I'd say I like your last suggestion, but maybe it should look like this:

@strawberry.subscription
async def time(self, info) -> strawberry.SubscriptionResult[str]:
    ...

so it looks more like a python typing type ;)

We should maybe also have a look at MyPy plugins, like the one for dataclasses: https://github.com/python/mypy/blob/master/mypy/plugins/dataclasses.py

[jaydenwindle]

I like that option too!

I'll have to look a little deeper into how MyPy does its magic, but I think if we define strawberry.SubscriptionResult as a custom type that's just a wrapper around typing.AsyncGenerator it should keep MyPy from complaining.