Minimal mypy update to openapy.py, adding dict type

[dave42w]

Description

This PR just fixes the missing dict type in 3 places so that mypy warnings are reduced.

PR Checklist

Please ensure that:

• [X ] The PR contains a descriptive title
• [X ] The PR contains a descriptive summary of the changes
• [X ] You build and test your changes before submitting a PR.
• [X ] You have added relevant documentation
• [X ] You have added relevant tests. We prefer integration tests wherever possible

Pre-Commit Instructions:

• [X ] Ensure that you have run the pre-commit hooks on your PR.

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
robyn	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Nov 23, 2024 0:28am

[sansyrox]

LGTM! Thanks @dave42w 😄

[codspeed-hq]

CodSpeed Performance Report

Merging #1048 will not alter performance

_{Comparing dave42w:mypy3 (7367060) with main (9a95611)}

Summary

✅ 146 untouched benchmarks

[dave42w]

Hey @sansyrox

I have a fix for the mypy errors with the way we are using TypedDict for variables eg

Variable "typing.TypedDict" is not valid as a type

the fix is a new class

class str_typed_dict(TypedDict):
    key : str
    value : str

which is then used like this:

        query_params: Optional[str_typed_dict],
        request_body: Optional[str_typed_dict],
        return_annotation: Optional[str_typed_dict],

and

    def get_openapi_type(self, typed_dict: str_typed_dict) -> str:

and tested like this:

            query_param_annotations = query_params.__annotations__ if query_params is str_typed_dict else typing.get_type_hints(query_params)

All tests pass and now we have no mypy warnings in OpenApi

Are you happy for me to commit this change?

[sansyrox]

@dave42w , yes ! Go ahead and thank you 😄

[dave42w]

@dave42w , yes ! Go ahead and thank you 😄

Done :-)

[sansyrox]

LGTM!