Closed austinvalle closed 7 months ago
I'm going to lock this pull request because it has been closed for 30 days β³. This helps our maintainers find and focus on the active contributions. If you have found a problem that seems related to this change, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.
Supersedes #109, #113
This PR contains a version bump from
github.com/pb33f/libopenapi v0.13.22
->v0.14.4
, along with all the fixes for breaking changes to the code structure. The original release notes can be found in the linked dependabot PR (#109) but I will summarize the effects ontfplugingen-openapi
here.Fixes from downstream
default
values are now properly being detected, where before they must have been not populated properly? I created a changelog for it, but the related OpenAPI spec's seem to match the behavior we have now ππ» . All 3 of the properties showing changes in the kubernetes test look the same:Changes from downstream
libopenapi
data model can be traversed in the exact order as defined in the OpenAPI spec, this manifests itself in a switch from using the Go built-inmap
, to a neworderedmap
type. The construction of this type in tests has changed as well as how you iterate over it.orderedmap
now requires acontext.Context
, which I have replaced with acontext.TODO
for now.(Parameter).Required
field changed frombool
to*bool
(Schema).Default
and(Schema).Enum
now use*yaml.Node
instead ofany
. Type assertions have been replaced with YAML unmarshaling/decoding.