-
-
Notifications
You must be signed in to change notification settings - Fork 91
v0.28.0 #472
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
v0.28.0 #472
Conversation
And security scheme deprecated field.
New `DataValue` and `SerializedValue`
I think this covers all of it according to the release notes and the migration guide.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR implements full and complete OpenAPI 3.2 support for libopenapi, adding all new OpenAPI 3.2 features while maintaining backward compatibility. The changes span across low-level data models, high-level APIs, change detection, and schema validation.
Key changes include:
- Addition of OpenAPI 3.2-specific fields across various objects (Server name, Response summary, XML nodeType, etc.)
- Support for new OpenAPI 3.2 features like additional operations, device OAuth flows, and mediaTypes components
- Implementation of change detection for all new 3.2 fields with appropriate breaking change classifications
- Updated schema validation to use the official OpenAPI 3.2 schema
Reviewed Changes
Copilot reviewed 68 out of 68 changed files in this pull request and generated 4 comments.
Show a summary per file
| File | Description |
|---|---|
| datamodel/schemas/oas32-schema.json | Updated to official OpenAPI 3.2 schema with proper versioning |
| datamodel/spec_info.go | Added $self field extraction for OpenAPI 3.2+ documents |
| datamodel/low/v3/*.go | Extended low-level models with OpenAPI 3.2 fields and proper hash computation |
| datamodel/high/v3/*.go | Added high-level wrappers for new OpenAPI 3.2 features |
| what-changed/model/*.go | Implemented change detection for all new 3.2 fields with breaking change analysis |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #472 +/- ##
========================================
Coverage 99.67% 99.68%
========================================
Files 171 171
Lines 24987 25472 +485
========================================
+ Hits 24906 25391 +485
Misses 76 76
Partials 5 5
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
tests shows code was not working as required, it is now.
additional operations were not being handled properly, now they are! use the raw key, or dump custom operation methods in with the known ones and libopenapi will pull them all out for you.
not sure why it’s failing. but anyway
works on my machine bro.
Full and complete OpenAPI 3.2 support.
https://learn.openapis.org/upgrading/v3.1-to-v3.2.html
https://github.com/OAI/OpenAPI-Specification/releases/tag/3.2.0
This is a breaking change
Issue #117 has been implemented which means
BuildV3Model()will return anerrorinstead of[]erroras we're now usingerrors.Join.