Skip to content

Support OpenAPI version 3.1.x on input#291

Merged
jplatte merged 1 commit intotamasfe:mainfrom
kw217:ignore-patch-version
Mar 17, 2026
Merged

Support OpenAPI version 3.1.x on input#291
jplatte merged 1 commit intotamasfe:mainfrom
kw217:ignore-patch-version

Conversation

@kw217
Copy link
Copy Markdown
Contributor

@kw217 kw217 commented Mar 5, 2026

https://spec.openapis.org/oas/v3.1.1.html#versions is clear that tools SHOULD ignore the patch version.

This was causing problems for us.

The fix just ignores the patch version, insisting only on 3.1.something.

@kw217
Copy link
Copy Markdown
Contributor Author

kw217 commented Mar 5, 2026

It wasn't immediately obvious to me how to add a test for this - let me know what the best approach would be. Also if you have a different preferred approach rather than regex. Thanks.

@jplatte
Copy link
Copy Markdown
Collaborator

jplatte commented Mar 5, 2026

Yeah I don't think it makes sense to introduce a dependency on regex for this. starts_with should be good enough, no?

@kw217 kw217 force-pushed the ignore-patch-version branch from 4a725ec to abdae8f Compare March 5, 2026 15:10
@kw217
Copy link
Copy Markdown
Contributor Author

kw217 commented Mar 5, 2026

Thanks, simplified :-)

@kw217
Copy link
Copy Markdown
Contributor Author

kw217 commented Mar 5, 2026

@jplatte I've fixed that now - please could you approve the workflow? This is my first contribution :-)

@jplatte jplatte merged commit b3e78aa into tamasfe:main Mar 17, 2026
2 checks passed
@kw217
Copy link
Copy Markdown
Contributor Author

kw217 commented Mar 18, 2026

Thanks!

@kw217 kw217 deleted the ignore-patch-version branch March 18, 2026 08:35
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