OpenAPI spec for Jupyter Workflows#531
OpenAPI spec for Jupyter Workflows#531nsingl00 wants to merge 2 commits intojupyter-server:dag-schedulingfrom
Conversation
nsingl00
force-pushed
the
workflow_openapi
branch
2 times, most recently
from
7d800f3 to
5d8fb4c
Compare
andrii-i
added
enhancement
nsingl00
force-pushed
the
workflow_openapi
branch
2 times, most recently
from
9ee87f8 to
0c60643
Compare
There was a problem hiding this comment.
Hi @nsingl00. Thank you for opening and rebasing this PR. Currently there are some .yaml formatting errors. I left suggestions on and comments about fixing them, could you please give them a look? Using IDE with .yaml linting should highlight all the errors mentioned.
Some OpenAPI tools that I find useful:
- VSCode OpenAPI plugin https://marketplace.visualstudio.com/items?itemName=42Crunch.vscode-openapi
- https://editor.swagger.io/, it can generate a (more) human-readable API docs which should make API discussions and reviews in the future easier. Please see https://app.swaggerhub.com/apis/andrii-i/jupyter-scheduler/2.7.1-oas3 for an example.
nsingl00
force-pushed
the
workflow_openapi
branch
2 times, most recently
from
70c342a to
4082eb9
Compare
andrii-i
left a comment
andrii-i
left a comment
There was a problem hiding this comment.
Thank you for the update @nsingl00. Currently some models / components referenced in the spec are missing from the components section: CreateJobFromDefinition, RuntimeEnvironment, DSNamespace, JobFile. Could you please add these missing models / components to the spec?
done |
nsingl00
force-pushed
the
workflow_openapi
branch
2 times, most recently
from
6349da0 to
73e4b2a
Compare
nsingl00
force-pushed
the
workflow_openapi
branch
from
af3c513 to
4eb10bc
Compare
for more information, see https://pre-commit.ci
2 participants
No description provided.