Skip to content
This repository was archived by the owner on Oct 24, 2023. It is now read-only.

Fixing JSON Schema Output #17

Merged
merged 7 commits into from
Jul 28, 2022
Merged

Fixing JSON Schema Output #17

merged 7 commits into from
Jul 28, 2022

Conversation

gregsdennis
Copy link
Member

A review of the changes I'd like to make to the output. Mostly notably covers json-schema-org/json-schema-spec#1249

@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Jul 25, 2022

Deploying with  Cloudflare Pages  Cloudflare Pages

Latest commit: 9d2c1e8
Status: ✅  Deploy successful!
Preview URL: https://1c0b9e44.blog-cie.pages.dev
Branch Preview URL: https://review-of-output.blog-cie.pages.dev

View logs

Copy link
Member

@Relequestual Relequestual left a comment

Choose a reason for hiding this comment

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

This is great story telling and context to show the changes you'd like to make and why. Hopefully this will allow us to get some more feedback!

@gregsdennis gregsdennis merged commit baf69f4 into main Jul 28, 2022
@gregsdennis gregsdennis deleted the review-of-output branch July 28, 2022 20:56
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants