Skip to content

Add JSON to Zod Schema converter#331

Merged
ritz078 merged 1 commit into
ritz078:masterfrom
csandman:json-to-zod
Jun 20, 2022
Merged

Add JSON to Zod Schema converter#331
ritz078 merged 1 commit into
ritz078:masterfrom
csandman:json-to-zod

Conversation

@csandman
Copy link
Copy Markdown
Contributor

@csandman csandman commented Jun 17, 2022

This PR is for a converter from JSON to a Zod schema, a popular TypeScript-first schema validation library.

If this is merged, I'd like to add other Zod converters, as there are many pre-built solutions for this mentioned in their ecosystem section.

These could include:

And potentially a few others.

One thing I wanted to make sure of, in the routes.tsx file, are GitHub or NPM links preferred for the packageUrl field? I saw that most of them are GitHub links so that's what I went with but I see that some are NPM so I wanted to make sure.

Besides that, let me know if you need any changes in order to merge!

@vercel
Copy link
Copy Markdown

vercel Bot commented Jun 17, 2022

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
transform ✅ Ready (Inspect) Visit Preview Jun 17, 2022 at 7:33PM (UTC)

Copy link
Copy Markdown
Owner

@ritz078 ritz078 left a comment

Choose a reason for hiding this comment

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

Great job 🎉

@ritz078
Copy link
Copy Markdown
Owner

ritz078 commented Jun 20, 2022

One thing I wanted to make sure of, in the routes.tsx file, are GitHub or NPM links preferred for the packageUrl field?

npm takes priority over GitHub.

@allcontributors please add @csandman for code contribution.

@allcontributors
Copy link
Copy Markdown
Contributor

@ritz078

I've put up a pull request to add @csandman! 🎉

@ritz078 ritz078 merged commit 0905e20 into ritz078:master Jun 20, 2022
@csandman csandman deleted the json-to-zod branch June 20, 2022 21:11
@csandman
Copy link
Copy Markdown
Contributor Author

npm takes priority over GitHub.

Ok sounds good, I updated the links in my newer PR to use NPM links instead!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants