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

Exclude BaseModel docstring from JSON schema description #8352

Merged
merged 5 commits into from Dec 12, 2023
Merged

Exclude BaseModel docstring from JSON schema description #8352

merged 5 commits into from Dec 12, 2023

Commits on Dec 11, 2023

  1. include default factory results as default in json schema

    @sydney-runkle
    sydney-runkle committed Dec 11, 2023
    Configuration menu
    Copy the full SHA
    29bb160 View commit details
    Browse the repository at this point in the history

  2. omit docstring of BaseModel from json_schema due to verbosity

    @sydney-runkle
    sydney-runkle committed Dec 11, 2023
    Configuration menu
    Copy the full SHA
    c9aedc4 View commit details
    Browse the repository at this point in the history

  3. Revert "include default factory results as default in json schema" 

    This reverts commit 29bb160.
    @sydney-runkle
    sydney-runkle committed Dec 11, 2023
    Configuration menu
    Copy the full SHA
    abcd94c View commit details
    Browse the repository at this point in the history

  4. using is BaseModel check instead of name check

    @sydney-runkle
    sydney-runkle committed Dec 11, 2023
    Configuration menu
    Copy the full SHA
    ed46ba2 View commit details
    Browse the repository at this point in the history

  5. adding test

    @sydney-runkle
    sydney-runkle committed Dec 11, 2023
    Configuration menu
    Copy the full SHA
    3494e77 View commit details
    Browse the repository at this point in the history