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

Fix misbehavior for models referencing redefined type aliases #8050

Merged
merged 2 commits into from Nov 7, 2023

Conversation

dmontagu
Copy link
Contributor

@dmontagu dmontagu commented Nov 7, 2023

This fixes some rare edge cases, e.g., if redefining a type alias over the course of a module, with some model including references to multiple reassignments of the alias. (See the added test for an example.)

@dmontagu dmontagu added the relnotes-fix Used for bugfixes. label Nov 7, 2023
@dmontagu dmontagu changed the title Add the id to generated type alias core refs Fix misbehavior for models referencing redefined type aliases Nov 7, 2023
Copy link

Deploying with  Cloudflare Pages  Cloudflare Pages

Latest commit: 54f7763
Status: ✅  Deploy successful!
Preview URL: https://5e925233.pydantic-docs2.pages.dev
Branch Preview URL: https://type-alias-ref-id.pydantic-docs2.pages.dev

View logs

@dmontagu dmontagu merged commit ecc18bd into main Nov 7, 2023
60 of 61 checks passed
@dmontagu dmontagu deleted the type-alias-ref-id branch November 7, 2023 19:22
@sydney-runkle sydney-runkle mentioned this pull request Nov 7, 2023
10 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
relnotes-fix Used for bugfixes.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants