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

Make HttpLlm.application() function convenient. #156

Merged
merged 1 commit into from
Mar 15, 2025
Merged

Conversation

samchon
Copy link
Owner

@samchon samchon commented Mar 15, 2025

This pull request includes changes to support additional OpenAPI and Swagger document versions in the HttpLlm and HttpMigration modules. The most important changes include updating the document type definitions and modifying the migration application composition process.

Support for additional document versions:

  • src/HttpLlm.ts: Added imports for OpenApiV3, OpenApiV3_1, and SwaggerV2 to support additional document versions. Updated the document type in the HttpLlm namespace to include these new types. [1] [2]
  • src/HttpMigration.ts: Added imports for OpenApiV3, OpenApiV3_1, and SwaggerV2 to support additional document versions. Updated the document type in the HttpMigration namespace to include these new types. [1] [2]

Migration application composition:

  • src/HttpLlm.ts: Modified the application method in the HttpLlm namespace to handle the updated document type without casting.
  • src/HttpMigration.ts: Modified the application method in the HttpMigration namespace to use OpenApi.convert for document conversion before composing the migration application.

@samchon samchon added the enhancement New feature or request label Mar 15, 2025
@samchon samchon self-assigned this Mar 15, 2025
Copy link
Owner Author

@samchon samchon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Test OK

@samchon samchon merged commit d08bb86 into master Mar 15, 2025
4 checks passed
@samchon samchon deleted the feat/props branch March 15, 2025 14:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant