Conversation
|
Thank you @stueynz! Checks are failing on lint, can you run |
Sure thing ... I'm assuming that warnings from things I haven't touched are ok. e.g explicit |
…alid-v2' to make it clear which version of spec the tests are for
…x) specs; Made validateSchema recursively check the entire object, not just the top level; check all items that could be in #/components/{schemas,parameters, requestBodies, responses}
…circular while checking things; validateSchema() is now recursive for non-circular APIs
|
Added a few more checks when doing validateSpec() - OpenAPI keeps what used to be in Also, when API doesn't have circular |
…de v14 stops spitting out warning in circular dependency
|
Finally, I've managed to unblock the release pipeline and we've got a stable test suite again. PHEW. Sadly we cannot use optional chaining, and I'm not entirely sure why. I think it might be a dependency of simplifyify being out of date or something, but nothing I can do on my end is getting past an error with lib/validation/spec.js having a unexpected . on line 42. |
|
Apologies for the long delay here - is this desired? I'm landing a few refactors and could land some of the spec validations under a major version bump. I'd be open to it, unless there's a different library that handles these changes that we should use internally instead. |
Re-did all my changes from last June/July into the top of main branch.
Hopefully can be picked up this time.
Added checks and validations for v3.0.x OpenAPI schemas to match the pre-existing Swagger (or v2.0) schema checks.
Added more known-errors - now that we're checking v3.0.x schemas, various real-world APIs are naughty.