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.
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
feat(codegen): add CLI to generate types given a codegen config #5982
feat(codegen): add CLI to generate types given a codegen config #5982
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
1 Ignored Deployment
|
No changes to documentation |
d0c1813
to
64e2e8d
Compare
0ed028b
to
e7cff51
Compare
Component Testing Report Updated Mar 19, 2024 3:30 PM (UTC)
|
64e2e8d
to
25bda14
Compare
e7cff51
to
4b1feae
Compare
25bda14
to
40225b3
Compare
4b1feae
to
1d3f323
Compare
40225b3
to
332270d
Compare
1d3f323
to
4d6e953
Compare
332270d
to
2073606
Compare
4d6e953
to
c33cbc7
Compare
2073606
to
7447899
Compare
c33cbc7
to
9fb7222
Compare
7447899
to
f111f34
Compare
9fb7222
to
e9db227
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you double check if the "workspace:*"
dependency is needed?
@@ -202,6 +202,7 @@ | |||
"@sanity/block-tools": "3.34.0", | |||
"@sanity/cli": "3.34.0", | |||
"@sanity/client": "^6.15.5", | |||
"@sanity/codegen": "workspace:*", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I believe we publish this package.json
file so it may be better to match the latest version of this package. I think pnpm/lerna should link/version it automatically. I don't think workspace:*
will resolve when publishing this to npm.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think workspace:* will resolve when publishing this to npm.
It will: https://pnpm.io/workspaces#publishing-workspace-packages (it should also be respected by lerna)
Ideally we should use the workspace protocol for all workspace packages.
resolver, | ||
}) | ||
|
||
for await (const result of queries) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nice
6405b6f
to
f92f7b4
Compare
84eede0
to
4ce884e
Compare
f92f7b4
to
b648872
Compare
4ce884e
to
0a18f59
Compare
b648872
to
92520dc
Compare
0a18f59
to
037d8b8
Compare
92520dc
to
4319333
Compare
037d8b8
to
f1da568
Compare
4319333
to
edb7f63
Compare
f1da568
to
4d9d54c
Compare
4d9d54c
to
f451330
Compare
Description
Adds a CLI command for generating types. This wraps up the interfaces appended in the previous PRs.
What to review
Testing
No specific tests added her as it's mainly about wrapping up interfaces from the other PRs.
Notes for release
N/A - no notes needed