-
Notifications
You must be signed in to change notification settings - Fork 244
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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update Hydrogen CLI to use latest cli-kit prompts #733
Conversation
This comment has been minimized.
This comment has been minimized.
Co-authored-by: Anthony Frehner <frehner@users.noreply.github.com>
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.
LGTM. I'm definitely not an expert in the cli area, so you may want to consider getting a good review from someone who is more familiar there. Up to you.
I think we're good to merge. If we keep waiting we'll create more conflicts with other CLI related PRs. |
Related: #397
This PR updates the CLI to use
@shopify/cli-kit@3.45.0
, which brings prompts backed by React 18. We could not do this earlier because the React version of the CLI had conflicts with the React installed in-app, but they are now compatible.馃帺 : Just play with all the CLI commands, especially
init
since that's the one using different prompts.Let's wait to merge this until3.45.0
is released as stable (right now we are on-pre.5
).