Skip to content

Conversation

@zzstoatzz
Copy link
Collaborator

@zzstoatzz zzstoatzz commented Mar 15, 2025

updates #17407 typing in new tests and removes (seemingly) unnecessary override of model_validate

Comment on lines -393 to -404
@classmethod
def model_validate(
cls: Type[Self],
obj: Any,
*,
strict: Optional[bool] = None,
from_attributes: Optional[bool] = None,
context: Optional[dict[str, Any]] = None,
) -> Self:
response = super().model_validate(obj)
return response

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not sure why this was here

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lol

@codspeed-hq
Copy link

codspeed-hq bot commented Mar 15, 2025

CodSpeed Performance Report

Merging #17490 will not alter performance

Comparing paginate-nits (28956c0) with paginate (29ddd40)

Summary

✅ 2 untouched benchmarks

@zzstoatzz zzstoatzz marked this pull request as ready for review March 15, 2025 22:27
@zzstoatzz zzstoatzz merged commit a73b0d9 into paginate Mar 16, 2025
48 checks passed
@zzstoatzz zzstoatzz deleted the paginate-nits branch March 16, 2025 03:03
zzstoatzz added a commit that referenced this pull request Mar 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants