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

Draft: Migrate to OpenAPI 3.1 #382

Draft
wants to merge 7 commits into
base: main
Choose a base branch
from

Conversation

philvarner
Copy link
Collaborator

Related Issue(s):

Proposed Changes:

  1. Update schemas to OpenAPI 3.1
  2. Clarify that the openapi30 conformance class from OAFeat shouldn't be used with these and remove from example landing pages
  3. add x-conformance-classes extension property to info section

PR Checklist:

  • This PR has no breaking changes.
  • This PR does not make any changes to the core spec in the stac-spec directory (these are included as a subtree and should be updated directly in radiantearth/stac-spec)
  • I have added my changes to the CHANGELOG or a CHANGELOG entry is not required.

@philvarner philvarner requested review from m-mohr and cholmes and removed request for m-mohr January 30, 2023 14:40
@philvarner philvarner changed the title Pv/openapi Migrate to OpenAPI 3.1 Jan 30, 2023
@m-mohr
Copy link
Collaborator

m-mohr commented Jan 30, 2023

I know I opened #31, but as OGC API seems still to use 3.0, I'm wondering whether we should stick to 3.0 for now to now mix versions and have less issues with tooling? I think 3.1 support in tooling is still evolving and we also don't really make use of any 3.1 improvements yet (i.e. JSON Schema instead of "OpenAPI schema") so there's no urgent reason to migrate, right?

The example -> examples change seems unrelated to 3.0 -> 3.1 and it also looks like examples is actually meant to be a Map, but I need to check that more closely....

@philvarner philvarner marked this pull request as draft January 31, 2023 00:22
@philvarner philvarner changed the title Migrate to OpenAPI 3.1 Draft: Migrate to OpenAPI 3.1 Jan 31, 2023
@philvarner
Copy link
Collaborator Author

Okay, I put this into draft and created a new PR for just the x-conformance-classes addition #385

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.

2 participants