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

OpenAPI + CI #86

Merged
merged 26 commits into from
Dec 8, 2020
Merged

OpenAPI + CI #86

merged 26 commits into from
Dec 8, 2020

Conversation

matthewhanson
Copy link
Collaborator

This is the 'dev' branch because I was using it to test the deployment of the dev versions of the OpenAPI files. Updates to dev will be immediately deployed to the dev/ folder in gh-pages

As in stac-spec, when a tag is created these files will be published to vXX folder.

Related Issue(s): #
#12

Proposed Changes:

  • bundle each openapi.yaml doc (resolving fragments)
  • publish yaml to gh-pages
  • create combined version of STAC + all extensions

@m-mohr
Copy link
Collaborator

m-mohr commented Dec 8, 2020

  1. Could we migrate the bash script to node to be platform independent? I can't run this on Windows...
  2. What's the plan regarding redoc?

@m-mohr m-mohr mentioned this pull request Dec 8, 2020
4 tasks
@matthewhanson
Copy link
Collaborator Author

@m-mohr I spent several hours trying to get a Node script working to do the build using speccy and in the end got sick of Promises and wrote that bash script in a few minutes. Just wanted to get it working.

Let's get this merged in so we have it building and then we can replace it with Node.

For Redoc we can just reference the published URLs from stacsite. I'm still figuring out the combine, there's a couple errors in the yaml that swagger-combine doesn't like, working on getting those resolved, but we can do a different PR for that.

@matthewhanson matthewhanson marked this pull request as ready for review December 8, 2020 19:48
@matthewhanson matthewhanson merged commit 5424bdf into master Dec 8, 2020
@m-mohr
Copy link
Collaborator

m-mohr commented Dec 8, 2020

I can do the Node script for beta.2 @matthewhanson

@m-mohr m-mohr deleted the dev branch December 8, 2020 20:39
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.

3 participants