Skip to content

Allow not subschemas#25

Merged
ahl merged 1 commit into
mainfrom
allow-not
Aug 14, 2025
Merged

Allow not subschemas#25
ahl merged 1 commit into
mainfrom
allow-not

Conversation

@ahl

@ahl ahl commented Aug 14, 2025

Copy link
Copy Markdown
Collaborator

Initiailly we didn't handle not constructions... mostly because we weren't sure about how they might come up, but also because we didn't have experience knowing how to interpret their meaning. Now they are coming up and we understand them well enough to treat them like other subschemas (i.e. "anyOf", "oneOf", "allOf").

Initiailly we didn't handle `not` constructions... mostly because we
weren't sure about how they might come up, but also because we didn't
have experience knowing how to interpret their meaning. Now they are
coming up **and** we understand them well enough to treat them like
other subschemas (i.e. "anyOf", "oneOf", "allOf").
@ahl ahl requested a review from sunshowers August 14, 2025 22:02

@sunshowers sunshowers left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

thanks!

@ahl ahl merged commit bdd1468 into main Aug 14, 2025
4 checks passed
@ahl ahl deleted the allow-not branch August 14, 2025 22:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants