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.

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

Openapi kt #763

Open
wants to merge 10 commits into
base: main
Choose a base branch
from
Open

Openapi kt #763

wants to merge 10 commits into from

Conversation

nomisRev
Copy link
Contributor

@nomisRev nomisRev commented Jun 28, 2024

This PR replaces the OpenAPI Generator using Mustache, with a Kotlin based generator. Which has support for a much wider range of complex operations, and types, and currently fully supports the OpenAI specification, and all OpenAPI types.

The changes in the PR are minimal:

  • nested schemas are now generated as nested models so CreateChatCompletionModel becomes CreateChatCompletion.Model.
  • There is no longer a difference between model, and api packages, simplifying imports of the generated code
  • Downloading the OpenAI OpenAPI Specification is refactored to Gradle Kotlin

A couple fixes need to be back ported still:

  • firstId and lastId should become non-required.
  • Serialisation Exception should include Response Info for logging

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