Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Improve error message for improper RootModel subclasses #8857

Merged
merged 5 commits into from Feb 20, 2024

Conversation

sydney-runkle
Copy link
Member

@sydney-runkle sydney-runkle commented Feb 20, 2024

Fix #8712

Improve error message in the case where RootModel isn't parametrized properly with generics.

Alternative to #8804.

Selected Reviewer: @samuelcolvin

@sydney-runkle sydney-runkle added the relnotes-change Used for changes to existing functionality which don't have a better categorization. label Feb 20, 2024
Copy link

cloudflare-pages bot commented Feb 20, 2024

Deploying with  Cloudflare Pages  Cloudflare Pages

Latest commit: 16e3989
Status: ✅  Deploy successful!
Preview URL: https://a9388c35.pydantic-docs2.pages.dev
Branch Preview URL: https://fix-rootmodel-gen-error.pydantic-docs2.pages.dev

View logs

Copy link

codspeed-hq bot commented Feb 20, 2024

CodSpeed Performance Report

Merging #8857 will not alter performance

Comparing fix_rootmodel_gen_error (16e3989) with main (5d09b47)

Summary

✅ 10 untouched benchmarks

@sydney-runkle
Copy link
Member Author

Please review

@sydney-runkle sydney-runkle merged commit 812516d into main Feb 20, 2024
54 checks passed
@sydney-runkle sydney-runkle deleted the fix_rootmodel_gen_error branch February 20, 2024 19:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready for review relnotes-change Used for changes to existing functionality which don't have a better categorization.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Weird error message on creating a generic RootModel
3 participants