Skip to content
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.

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

Add support for composable schemas in zed validate #478

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

tstirrat15
Copy link
Contributor

Description

Part of composable schema support. We want to make it so that zed validate can operate on both composable schemas and standard schemas.

This implements it by adding a new behavior and a new flag:

  • By default, validate will attempt to validate a schema first according to composable schema rules, then standard schema rules, and if both fail it will show the errors from composable schema.
  • There's a new flag: schema-syntax. If you add --schema-syntax composable, it will validate according to only composable schema syntax rules, and if you add --schema-syntax standard, it will validate according to only standard schema rules. The default value for this flag is the empty string.

I also added proper tests for the validate command, which I'd neglected up until now.

Changes

TODO

Testing

TODO

@tstirrat15 tstirrat15 force-pushed the OSS-214/composable-schema-validation branch from 779c4a8 to a4e4f6f Compare March 11, 2025 23:03
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.

1 participant