Thanks to visit codestin.com
Credit goes to github.com

Skip to content
This repository was archived by the owner on Apr 11, 2026. It is now read-only.

Relax intensity of error for issubclass and isinstance schemas from json#1479

Merged
sydney-runkle merged 4 commits into
mainfrom
lax-internal-errs
Oct 10, 2024
Merged

Relax intensity of error for issubclass and isinstance schemas from json#1479
sydney-runkle merged 4 commits into
mainfrom
lax-internal-errs

Conversation

@sydney-runkle

Copy link
Copy Markdown
Contributor

Closes pydantic/pydantic#8890

Makes these more compatible with validation in unions.

@codspeed-hq

codspeed-hq Bot commented Oct 9, 2024

Copy link
Copy Markdown

CodSpeed Performance Report

Merging #1479 will not alter performance

Comparing lax-internal-errs (68f000d) with main (dc4846e)

Summary

✅ 155 untouched benchmarks

@davidhewitt davidhewitt left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

The overall motivation of the change looks good, though I wonder about an alternative naming?

Comment thread src/validators/is_instance.rs

@davidhewitt davidhewitt left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Looks good to me!

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

NotImplementedError: Cannot check isinstance when validating from json, use a JsonOrPython validator instead.

2 participants