Patrick Arminio
Patrick Arminio
> The issue with making sync resolvers magically async is that graphql potentially would have to resolve thousands of them per single request 🤔, that's why I think it's inefficient...
interesting, I think it might be also worth doing tests with a database 😊 We should probably have a test suite somewhere for this, what do you think?
@jgadling thanks for the suggestion! I think that would be neat, but I'd prefer to keep the base dependencies at a minimum if possible 😊
@RonquilloAeon would have some time to make a tiny reproduction? that will help fixing this issue :D
@jgadling that's pretty cool! would be interested in having this into Strawberry?
judging from the reaction on this issue I think it is! I guess ideally we could have a `strawberry.fastapi.field` in future, but I think just having the extension for now...
> Ideally not FastAPI specific! If there's a way to mark arguments in general as not inputs/fields (as in the first comment) that would be amazing. This means we could...
@cadlagtrader just double checked hits and I don't get the same error do you have any custom settings for Mypy?
that also works for me, would you be able to make a tiny repo with this issue?
yup, we really should do #2687, that would make things quite a bit simpler for users I think