-
Notifications
You must be signed in to change notification settings - Fork 35
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
Openapi Workflow #797
Openapi Workflow #797
Conversation
acarnagey
commented
Jun 2, 2023
•
edited
Loading
edited
- Adds workflows to openapi paths
- Refactor paths into separate files
@acarnagey , could you add some more comments on this PR, pls? It isn't entirely clear on the call what is the effects of this. |
@acarnagey, suggestion to split this into two separate PRs. |
Also, @acarnagey, we recently removed the RDF classes from the respec: Can you please remove that in the PR? |
…napi-workflows # Conflicts: # docs/openapi/openapi.yml
Do not merge need to fix script that generates openapi.yml on npm i |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Mark as draft, make CI pass, request a re-review
@@ -8,11 +8,22 @@ const yaml = require('js-yaml'); | |||
const Ajv = require('ajv').default; | |||
const addFormats = require('ajv-formats').default; | |||
|
|||
const schemaKeys = ['common', 'credentials', 'presentations', 'snippets']; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
these are "components", but that is a nit
Closing this one out, the allOf in paths was causing additional validation errors |