Skip to content

Fix typo in docs for new private dunder pydantic method names#7657

Merged
hramezani merged 1 commit intopydantic:mainfrom
austinorr:Patch-docs-typo-in-dunder-method-names
Sep 27, 2023
Merged

Fix typo in docs for new private dunder pydantic method names#7657
hramezani merged 1 commit intopydantic:mainfrom
austinorr:Patch-docs-typo-in-dunder-method-names

Conversation

@austinorr
Copy link
Copy Markdown
Contributor

@austinorr austinorr commented Sep 27, 2023

Change Summary

Relatively trivial docs change to indicate new naming convention for dunder methods. Current docs don't match the methods listed in the table since the regex in the narrative includes more wildcards than the actually used convention.

  • My PR is ready to review

Selected Reviewer: @davidhewitt

@austinorr
Copy link
Copy Markdown
Contributor Author

please review

@hramezani hramezani added the relnotes-ignore Omit this PR from the release notes. label Sep 27, 2023
@hramezani
Copy link
Copy Markdown
Member

Thanks @austinorr

@hramezani hramezani merged commit 24e9222 into pydantic:main Sep 27, 2023
@samuelcolvin
Copy link
Copy Markdown
Member

I'm not sure this is really correct - we have __get_pydantic_core_schema__.

@hramezani
Copy link
Copy Markdown
Member

I'm not sure this is really correct - we have __get_pydantic_core_schema__.

Ah, you are right. just created a revert PR #7662

Sorry for that!

@austinorr austinorr deleted the Patch-docs-typo-in-dunder-method-names branch September 28, 2023 23:08
@austinorr
Copy link
Copy Markdown
Contributor Author

I'm not sure this is really correct - we have __get_pydantic_core_schema__.

Ah, you are right. just created a revert PR #7662

Sorry for that!

My mistake all, thanks for the quick responses and very sorry for the extra noise.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready for review relnotes-ignore Omit this PR from the release notes.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants