Skip optimised check when returning generic types

[patrick91]

Closes #2954

There's probably a better way to do this :) but I'm happy to do that in another PR.

[codecov]

Codecov Report

Merging #2955 (892e4d5) into main (18f8b44) will increase coverage by 0.00%.
The diff coverage is 100.00%.

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #2955   +/-   ##
=======================================
  Coverage   96.11%   96.11%           
=======================================
  Files         211      211           
  Lines        9242     9244    +2     
  Branches     1491     1492    +1     
=======================================
+ Hits         8883     8885    +2     
  Misses        229      229           
  Partials      130      130           

[patrick91]

One way to make this better could be to add some metadata when doing things like this: Real[int](y=0) (class_getitem could use that)

but it won't fix the cases where some one does Real(y=0), I'm sure we have some code that checks the right type somewhere though 😊

[botberry]

Thanks for adding the RELEASE.md file!

Here's a preview of the changelog:

---

This release fixes a bug where returning a generic type from a field
that was returning an interface would throw an error.

---

Here's the preview release card for twitter:

Here's the tweet text:

🆕 Release (next) is out! Thanks to @patrick91 for the PR 👏

Get it here 👉 https://github.com/strawberry-graphql/strawberry/releases/tag/(next)

[codspeed-hq]

CodSpeed Performance Report

Merging #2955 will not alter performance

_{Comparing interface-missing-type (892e4d5) with main (18f8b44)}

Summary

✅ 5 untouched benchmarks