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

breaking change to getkin-openapi in minor version 0.122 from 0.120 #324

Closed
genslein opened this issue Jan 18, 2024 · 2 comments
Closed

breaking change to getkin-openapi in minor version 0.122 from 0.120 #324

genslein opened this issue Jan 18, 2024 · 2 comments

Comments

@genslein
Copy link
Contributor

getkin/kin-openapi#763

Breaking change was introduced in a minor version. Don't upgrade getkin unless vervet has been upgraded to deal with this
@jgresty
Copy link
Member

jgresty commented Jan 18, 2024

I've started a branch to upgrade it: feat/update-kin-openapi

That branch changes all the references so it compiles, however we are panicking at runtime when resolving references:

panic: reflect: reflect.Value.Set using value obtained using unexported field

If anyone wants to figure out why then go ahead.

@tinygrasshopper
Copy link
Contributor

kin-openapi upgraded to latest in recent releases of vervet

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@tinygrasshopper @genslein @jgresty