Fix "Failing to pass a value to the type_params parameter of typing._eval_type"

[patrick91]

Closes #3690

Summary by Sourcery

Bug Fixes:

• Fix the issue where the 'type_params' parameter was not being passed to the '_eval_type' function in 'strawberry/utils/typing.py'.

[sourcery-ai]

Reviewer's Guide by Sourcery

This PR fixes a bug in the type evaluation functionality by explicitly passing None as the type_params parameter to the _eval_type function call. The change ensures proper handling of forward references in type evaluation.

Sequence diagram for type evaluation with _eval_type

sequenceDiagram
    participant eval_type
    participant _eval_type

    eval_type->>_eval_type: Call _eval_type(type_, globalns, localns, type_params=None)
    _eval_type-->>eval_type: Return evaluated type

    note right of eval_type: Explicitly passing None to type_params

Loading

File-Level Changes

Change	Details	Files
Added missing type_params parameter in _eval_type function call	Updated _eval_type function call to include type_params=None parameter Fixed potential error when evaluating ForwardRef types	strawberry/utils/typing.py

Assessment against linked issues

Issue	Objective	Addressed	Explanation
#3690	Fix the DeprecationWarning about missing type_params parameter in typing._eval_type	✅

Possibly linked issues

• DeprecationWarning: Failing to pass a value to the type_params parameter of typing._eval_type #3690: The PR fixes the DeprecationWarning by passing type_params=None to _eval_type.

---

Tips and commands

Interacting with Sourcery

• Trigger a new review: Comment @sourcery-ai review on the pull request.
• Continue discussions: Reply directly to Sourcery's review comments.
• Generate a GitHub issue from a review comment: Ask Sourcery to create an
  issue from a review comment by replying to it.
• Generate a pull request title: Write @sourcery-ai anywhere in the pull
  request title to generate a title at any time.
• Generate a pull request summary: Write @sourcery-ai summary anywhere in
  the pull request body to generate a PR summary at any time. You can also use
  this command to specify where the summary should be inserted.

Customizing Your Experience

Access your dashboard to:

• Enable or disable review features such as the Sourcery-generated pull request
  summary, the reviewer's guide, and others.
• Change the review language.
• Add, remove or edit custom review instructions.
• Adjust other review settings.

Getting Help

• Contact our support team for questions or feedback.
• Visit our documentation for detailed guides and information.
• Keep in touch with the Sourcery team by following us on X/Twitter, LinkedIn or GitHub.

[sourcery-ai]

Hey @patrick91 - I've reviewed your changes and they look great!

Here's what I looked at during the review

• 🟢 General issues: all looks good
• 🟢 Security: all looks good
• 🟢 Testing: all looks good
• 🟢 Complexity: all looks good
• 🟢 Documentation: all looks good

---

Sourcery is free for open source - if you like our reviews please consider sharing them ✨

• X
• Mastodon
• LinkedIn
• Facebook

_{Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.}

[botberry]

Thanks for adding the RELEASE.md file!

Here's a preview of the changelog:

---

This release fixes the following deprecation warning:

Failing to pass a value to the 'type_params' parameter of 'typing._eval_type' is deprecated,
as it leads to incorrect behaviour when calling typing._eval_type on a stringified annotation
that references a PEP 695 type parameter. It will be disallowed in Python 3.15.

This was only trigger in Python 3.13 and above.

Here's the tweet text:

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

Get it here 👉 https://strawberry.rocks/release/(next)

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 97.01%. Comparing base (04936fd) to head (e03b32c).
Report is 1 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #3692   +/-   ##
=======================================
  Coverage   97.01%   97.01%           
=======================================
  Files         502      502           
  Lines       33568    33571    +3     
  Branches     5634     5635    +1     
=======================================
+ Hits        32565    32568    +3     
  Misses        796      796           
  Partials      207      207           

[codspeed-hq]

CodSpeed Performance Report

Merging #3692 will not alter performance

_{Comparing feature/fix-deprecation (e03b32c) with main (04936fd)}

Summary

✅ 15 untouched benchmarks