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

Split swagger.yml into separate files #185

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

Conversation

maxno-kivra
Copy link
Member

OpenAPI has support for multiple files using $ref. This uses that to split off the partner API into separate files. To make it easy to find the files I'm nameing them exactly like the path in the main swagger file.

There's one catch though. The cloudbuild has to be updated to include all the new files, and to not add to the pile of one-liners I'm moving the bash script into its own file. This has a caveat in that Cloud Build uses substitutions instead of environment variables. The fix is easy enough, pass BRANCH_NAME as an explicit argument to the script.

OpenAPI has support for multiple files using `$ref`. This uses that to split off
the partner API into separate files. To make it easy to find the files I'm
nameing them exactly like the `path` in the main swagger file.

There's one catch though. The cloudbuild has to be updated to include all the
new files, and to not add to the pile of one-liners I'm moving the bash script
into its own file. This has a caveat in that Cloud Build uses _substitutions_
instead of environment variables. The fix is easy enough, pass BRANCH_NAME as an
explicit argument to the script.
@maxno-kivra maxno-kivra self-assigned this Oct 24, 2024
@maxno-kivra maxno-kivra requested a review from a team October 25, 2024 07:34
@maxno-kivra maxno-kivra marked this pull request as draft October 28, 2024 13:25
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.

1 participant