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

WIP on #230 coming from #318 #412

Draft
wants to merge 19 commits into
base: master
Choose a base branch
from
Draft

Conversation

fenollp
Copy link
Collaborator

@fenollp fenollp commented Aug 26, 2021

No description provided.

Signed-off-by: Pierre Fenoll <[email protected]>
Signed-off-by: Pierre Fenoll <[email protected]>
Signed-off-by: Pierre Fenoll <[email protected]>
Signed-off-by: Pierre Fenoll <[email protected]>
Signed-off-by: Pierre Fenoll <[email protected]>
@fenollp fenollp marked this pull request as draft August 26, 2021 19:32
Signed-off-by: Pierre Fenoll <[email protected]>
Signed-off-by: Pierre Fenoll <[email protected]>
Signed-off-by: Pierre Fenoll <[email protected]>
Signed-off-by: Pierre Fenoll <[email protected]>
Signed-off-by: Pierre Fenoll <[email protected]>
Signed-off-by: Pierre Fenoll <[email protected]>
Signed-off-by: Pierre Fenoll <[email protected]>
Signed-off-by: Pierre Fenoll <[email protected]>
Signed-off-by: Pierre Fenoll <[email protected]>
Signed-off-by: Pierre Fenoll <[email protected]>
Signed-off-by: Pierre Fenoll <[email protected]>
Signed-off-by: Pierre Fenoll <[email protected]>
@scotchneat
Copy link

This comment suggests the data validation logic is getting rewritten in this PR.
Stoplight/prism structures their violations nicely which makes straightforward to parse. It would be nice to get something like that.

The last commit to this PR was 8 months ago and it's still a draft. Is there an update on what the plan is for this?

@fenollp
Copy link
Collaborator Author

fenollp commented Mar 24, 2022

@scotchneat #487 should be the way to go, once I find time to review it and move it forward.

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