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.

Sign up for GitHub

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

Jump to bottom

Add OpenAPI 3.1 support #590

Open
wants to merge 15 commits into
base: master
Choose a base branch
from
Open

Add OpenAPI 3.1 support #590

wants to merge 15 commits into from

Conversation

m-burgoyne
Copy link
Collaborator

No description provided.

@m-burgoyne m-burgoyne added the API EDR V1.2 Non-breaking change for Version 1.2 label Jan 6, 2025
chris-little
chris-little approved these changes Jan 8, 2025
View reviewed changes
Copy link
Contributor

@chris-little chris-little left a comment •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good work to get all this in. some editorial changes can be fixed later: spatiotemporal or spatio-temporal, or lower case 'shall' in test annex, or HTTP vs HTTPS.

And title page dates and version.

chris-little
chris-little reviewed Jan 9, 2025
View reviewed changes
LICENSE Outdated Show resolved Hide resolved
chris-little
chris-little reviewed Jan 9, 2025
View reviewed changes
README.md Outdated Show resolved Hide resolved
chris-little
chris-little reviewed Jan 9, 2025
View reviewed changes
core/standard/sections/annex_ats.adoc Outdated
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

o sure about this SHALL

chris-little
chris-little reviewed Jan 9, 2025
View reviewed changes
docs/AgreedDevelopmentProcesses.md Outdated
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We should leave this as shall try to follow, or change to should follow

chris-little
chris-little reviewed Jan 9, 2025
View reviewed changes
extensions/pubsub/standard/sections/clause_0_front_material.adoc Outdated
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think we should leave the OGC Boiler plate text 'as is

chris-little
chris-little reviewed Jan 9, 2025
View reviewed changes
proposals/README.md Outdated
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do we want to tie ourselves strictly to this process? Perhaps shall or even should?

chris-little
chris-little approved these changes Jan 9, 2025
View reviewed changes
Copy link
Contributor

@chris-little chris-little left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Replaced a few SHALLs by shalls

@chris-little
Copy link
Contributor

EDR API SWG 126 2025-01-09 agreed to merge next week, after further reviews.
@tomkralidis @ShaneMill1 @solson-nws @dblodgett-usgs @iandruska-ibl

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@chris-little chris-little chris-little approved these changes

@tomkralidis tomkralidis Awaiting requested review from tomkralidis

@dblodgett-usgs dblodgett-usgs Awaiting requested review from dblodgett-usgs

@solson-nws solson-nws Awaiting requested review from solson-nws

@ShaneMill1 ShaneMill1 Awaiting requested review from ShaneMill1

@iandruska-ibl iandruska-ibl Awaiting requested review from iandruska-ibl

Assignees
No one assigned
Labels
API EDR V1.2 Non-breaking change for Version 1.2
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@m-burgoyne @chris-little