Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

feat: support offline templates #10

Merged
merged 1 commit into from
Jan 5, 2025
Merged

feat: support offline templates #10

merged 1 commit into from
Jan 5, 2025

Conversation

kemingy
Copy link
Owner

@kemingy kemingy commented Jan 5, 2025

No description provided.

Signed-off-by: Keming <kemingy94@gmail.com>
@kemingy kemingy requested a review from Copilot January 5, 2025 17:10
@kemingy kemingy enabled auto-merge (squash) January 5, 2025 17:10
@kemingy kemingy merged commit 5158878 into main Jan 5, 2025
1 check passed
@kemingy kemingy deleted the offline branch January 5, 2025 17:10

Choose a reason for hiding this comment

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

Copilot reviewed 5 out of 5 changed files in this pull request and generated no comments.

Comments suppressed due to low confidence (3)

examples/offline_openapi.py:64

  • Ensure that the new behavior introduced in OpenAPIRender is covered by tests.
class OpenAPIRender:

pyproject.toml:4

  • The description should be 'OpenAPI spec generation and web page rendering.'
description = "OpenAPI spec generation and web page rendering."

defspec/server.py:21

  • [nitpick] The method name 'build' is ambiguous. It should be renamed to something more descriptive like 'set_openapi_handler'.
def build(cls, spec: bytes, path: str = "/openapi") -> Self:
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant