restful-react icon indicating copy to clipboard operation
restful-react copied to clipboard

Move openapi validator to speccy

Open fabien0102 opened this issue 6 years ago • 7 comments

Is your feature request related to a problem? Please describe.

The ibm validator is nice, but way to strict… no un-usuable in practice (for example type: uuid is an error), let's move to speccy from our old friends -> wework!

https://github.com/wework/speccy

Describe the solution you'd like We need to way to enable/disable the validation and specify some custom rules from restful-react generate options.

Describe alternatives you've considered https://github.com/IBM/openapi-validator already in place, but sadly always disabled because it's too strict…

Additional context The main idea to have a validator/linter part of our tool is to provide a better message when specs are not valid. Restful-react assume that the spec is perfectly valid for now, so it can break with a non helping message… (at least it can be better)

Also, our backend team already use speccy as validator in CI 😉

fabien0102 avatar Jul 25 '19 08:07 fabien0102

100% support this.

TejasQ avatar Oct 28 '19 10:10 TejasQ

Any thoughts on upgrading openapi-validator from 0.16.0 to 0.46.4?

marckassay avatar Aug 13 '21 17:08 marckassay

Any thoughts on upgrading openapi-validator from 0.16.0 to 0.46.4?

You can! Just open a PR and I will review it with pleasure 😁

fabien0102 avatar Aug 13 '21 19:08 fabien0102

And so can you. But why would or wouldn't you?

marckassay avatar Aug 14 '21 16:08 marckassay

I can do it, I was just in vacation last week, so not in front of a computer 😉

fabien0102 avatar Aug 16 '21 11:08 fabien0102

Any thoughts on upgrading openapi-validator from 0.16.0 to 0.46.4?

Done, should be on 15.9.0 release 😀

fabien0102 avatar Aug 16 '21 11:08 fabien0102

@fabien0102 Oh, thanks! I appreciate your work here on this project.

marckassay avatar Aug 17 '21 11:08 marckassay