Skip to content

Make generic TypeAliasType specifications produce different schema definitions #7893

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.

Sign up for GitHub

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

Jump to bottom
Merged
sydney-runkle merged 7 commits into from
Oct 26, 2023
Merged

Make generic TypeAliasType specifications produce different schema definitions #7893

sydney-runkle merged 7 commits into from
Oct 26, 2023

Conversation

@alexdrydew
Copy link
Contributor

@alexdrydew alexdrydew commented Oct 21, 2023 •
edited by pydantic-hooky bot
Loading

Change Summary

This PR fixes #7840: now different specifications of a generic TypeAliasType produce different definitions in pydantic schema which prevents "overwrites" of other specifications with the same origin.

Related issue number

Checklist

  • The pull request title is a good summary of the changes - it will be used in the changelog
  • Unit tests for the changes exist
  • Tests pass on CI
  • Documentation reflects the changes where applicable
  • My PR is ready to review, please add a comment including the phrase "please review" to assign reviewers

Selected Reviewer: @sydney-runkle

@alexdrydew alexdrydew marked this pull request as ready for review October 21, 2023 15:13
@alexdrydew
Copy link
Contributor Author

alexdrydew commented Oct 21, 2023

please review

@davidhewitt davidhewitt added the relnotes-fix Used for bugfixes. label Oct 24, 2023
sydney-runkle
sydney-runkle suggested changes Oct 25, 2023
View reviewed changes
Copy link
Contributor

@sydney-runkle sydney-runkle left a comment

Choose a reason for hiding this comment

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

LGTM overall, left a few nit picky change requests.

Thanks for the contribution ⭐!!

@pydantic-hooky pydantic-hooky bot added awaiting author revision awaiting changes from the PR author and removed ready for review labels Oct 25, 2023
@pydantic-hooky pydantic-hooky bot assigned alexdrydew and unassigned sydney-runkle Oct 25, 2023
dmontagu
dmontagu reviewed Oct 25, 2023
View reviewed changes
dmontagu
dmontagu reviewed Oct 25, 2023
View reviewed changes
@dmontagu dmontagu mentioned this pull request Oct 25, 2023
Merged
sydney-runkle
sydney-runkle reviewed Oct 25, 2023
View reviewed changes
sydney-runkle
sydney-runkle approved these changes Oct 26, 2023
View reviewed changes
Copy link
Contributor

@sydney-runkle sydney-runkle left a comment

Choose a reason for hiding this comment

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

@alexdrydew,

Thanks for the contribution 🌟 ! Looks great.

@sydney-runkle sydney-runkle added ready for review and removed awaiting author revision awaiting changes from the PR author labels Oct 26, 2023
@sydney-runkle sydney-runkle enabled auto-merge (squash) October 26, 2023 19:09
@sydney-runkle sydney-runkle merged commit ebecf91 into pydantic:main Oct 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@dmontagu dmontagu dmontagu left review comments

+1 more reviewer

@sydney-runkle sydney-runkle sydney-runkle approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@alexdrydew alexdrydew

Labels

ready for review relnotes-fix Used for bugfixes.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

PEP 695 generic type alias does not work

4 participants

@alexdrydew @dmontagu @sydney-runkle @davidhewitt