-
Notifications
You must be signed in to change notification settings - Fork 997
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
CRWA: Prompt for installation dir #8955
Conversation
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 from a telemetry perspective. I tested locally and could see the new error state being recorded and from looking at the changes I can't see how this would break anything else.
@Tobbe I testing this locally just by building the package and then running If I ran Are you able to confirm this? I think we'd aim for consistency here right or do we want it to be different? |
Ahh, yes. That's because your shell is doing path expansion if you pass it as an argument. But apparently that doesn't happen when it's just a prompt answer |
@Josh-Walker-GM Confirmed. Node does not support ~ on its own nodejs/node#684 |
Ahh so obvious now you say it! I forget |
I'm not so sure. I think we might still want to support it. Just need to figure out how! As you said - I think users would expect feature parity between cli args and prompt answers |
@Josh-Walker-GM Please try again. |
Great works for me too! |
Instead of failing if the user didn't provide a dir on the command line, let's prompt for one! I was inspired by t3 where you don't have to specify any arguments at all to get started ![image](https://github.com/redwoodjs/redwood/assets/30793/575775c1-555e-4e0f-851a-cd2bad49b5a9) This is what our implementation looks like now Default suggestion `my-redwood-app` ![image](https://github.com/redwoodjs/redwood/assets/30793/738856e9-e6f3-436d-bb56-a74ffc44ead2) And here's when it's done ![image](https://github.com/redwoodjs/redwood/assets/30793/55893785-52e3-41a2-9b75-1f88c4380293) Here's with a couple of options specified as arguments/flags ![image](https://github.com/redwoodjs/redwood/assets/30793/df65951c-b8aa-41f4-b586-11121f20a2be) @Josh-Walker-GM Can you please double check I didn't mess anything up with Telemetry? @ahaywood Hope I'm not stepping on your toes here 🦶
Instead of failing if the user didn't provide a dir on the command line, let's prompt for one!
I was inspired by t3 where you don't have to specify any arguments at all to get started
This is what our implementation looks like now
Default suggestion
my-redwood-app
And here's when it's done
Here's with a couple of options specified as arguments/flags
@Josh-Walker-GM Can you please double check I didn't mess anything up with Telemetry?
@ahaywood Hope I'm not stepping on your toes here 🦶