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

Schema validation #299

Merged
merged 10 commits into from Oct 24, 2017
Merged

Schema validation #299

merged 10 commits into from Oct 24, 2017

Conversation

bjoerge
Copy link
Member

@bjoerge bjoerge commented Oct 24, 2017

This provides a first iteration of better schema validation.

It is based on runtime schema information, and there's currently no way to infer file/line numbers.

There are also quite a few areas of improvements (and the implementation is a bit messy, sorry about that), but it should work as a first version that can be expanded on later.

@bjoerge bjoerge requested a review from rexxars October 24, 2017 11:14
@bjoerge bjoerge merged commit df520df into next Oct 24, 2017
@bjoerge bjoerge deleted the feature/schema-validation branch October 24, 2017 15:12
bjoerge added a commit that referenced this pull request Oct 25, 2017
* [schema] Add a new way to traverse/prepare schema and basic validation infrastructure

* [base] Expose validation groups on result of createSchema

* [default-layout] Go into 'failure mode' and display schema errors if any

* [test-studio] Remove object without fields test case as its now a hard error

* [code-input] Fix schema warning

* [default-layout] Styling the schema error screen

* [schema] Clean up help ids

* [schema] Error message improvements + reserve a few more type names

* [schema] More help-id cleanup

* [schema] Register reference validator
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.

None yet

2 participants