Skip to content

Switch to OAPI for codegen#5

Merged
mattoni merged 5 commits intomainfrom
mattoni/test-oapi-codegen
Feb 7, 2025
Merged

Switch to OAPI for codegen#5
mattoni merged 5 commits intomainfrom
mattoni/test-oapi-codegen

Conversation

@mattoni
Copy link
Copy Markdown
Member

@mattoni mattoni commented Feb 7, 2025

Switching this for a few reasons

  • More idiomatic go, and a lot less magic that was happening in ogen
  • As a result of less magic, there are less esoteric errors when generating code from the spec that may not work with the magic. Discriminated unions in ogen are picky, not as much here.
  • Seems this library is most likely to be supporting 3.1.0 first, open discussions on adopting libopenapi under the hood which should help with future versions too.

Still need to downconvert the OpenAPI spec for now, but it's acceptable for now.

@mattoni mattoni requested a review from blewisCycle February 7, 2025 23:18
@mattoni mattoni merged commit 1979134 into main Feb 7, 2025
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.

1 participant