Skip to content

Update JSON Schema documentation for V2#6492

Merged
tpdorsey merged 7 commits intomainfrom
json-schema-docs
Jul 10, 2023
Merged

Update JSON Schema documentation for V2#6492
tpdorsey merged 7 commits intomainfrom
json-schema-docs

Conversation

@tpdorsey
Copy link
Copy Markdown
Contributor

@tpdorsey tpdorsey commented Jul 6, 2023

Change Summary

Related issue number

Closes PYD-116

Checklist

  • Unit tests for the changes exist
  • Tests pass on CI and coverage remains at 100%
  • Documentation reflects the changes where applicable
  • changes/<pull request or issue id>-<github username>.md file added describing change
    (see changes/README.md for details)
  • My PR is ready to review, please add a comment including the phrase "please review" to assign reviewers

Selected Reviewer: @adriangb

@linear
Copy link
Copy Markdown

linear Bot commented Jul 6, 2023

@cloudflare-workers-and-pages
Copy link
Copy Markdown

cloudflare-workers-and-pages Bot commented Jul 6, 2023

Deploying with  Cloudflare Pages  Cloudflare Pages

Latest commit: f4009f0
Status: ✅  Deploy successful!
Preview URL: https://2dff25f6.pydantic-docs2.pages.dev
Branch Preview URL: https://json-schema-docs.pydantic-docs2.pages.dev

View logs

Comment thread docs/usage/json_schema.md Outdated
Comment thread docs/usage/json_schema.md
tpdorsey and others added 2 commits July 7, 2023 08:53
Co-authored-by: Marcelo Trylesinski <marcelotryle@gmail.com>
@tpdorsey
Copy link
Copy Markdown
Contributor Author

tpdorsey commented Jul 7, 2023

please review

Copy link
Copy Markdown
Member

@adriangb adriangb left a comment

Choose a reason for hiding this comment

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

left some notes but generally looks good!

Comment thread docs/usage/json_schema.md Outdated
Comment thread docs/usage/json_schema.md
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants