Skip to content

Conversation

Viicos
Copy link
Member

@Viicos Viicos commented Nov 10, 2023

Change Summary

Had to fetch new typeshed changes in #6563, and some unrelated errors got catched by latest pyright.

Not really satisfied with issues with Self (ideally return types should be Self | FieldInfo, but well..)

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

@Viicos
Copy link
Member Author

Viicos commented Nov 10, 2023

Please review

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, thanks @Viicos for all of your help this week 🌟

@sydney-runkle sydney-runkle added the relnotes-packaging Used for dependency changes. label Nov 10, 2023
@sydney-runkle sydney-runkle merged commit 2358e63 into pydantic:main Nov 11, 2023
@Viicos Viicos deleted the update-pyright branch November 11, 2023 13:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready for review relnotes-packaging Used for dependency changes.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants