Skip to content
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]: Switch to using enquirer, add engine compatibility override option #6802

Conversation

redwoodjs-bot[bot]
Copy link

@redwoodjs-bot redwoodjs-bot bot commented Nov 5, 2022

Hey All,

This should be considered a draft, I want to do some more testing. But I drafted what CRWA could look like with a switch over to enquirer. I think this sets a basis for future improvements really nicely.

One area I ran into issues was with the engine checks. I was hoping to only have one Listr, however had to keep it split out into it's own seperate Listr process because of the bug with overflowing terminal height and text display (see listr2/listr2#296).

Once @jtoar moves other areas of the codebase over to enquirer I can use this as a basis for the main PR (#6380) . But maybe we don't need to wait?

…ption (#6723)

* Add engine check and prompt

* Language tweak

* Add link and additional text to error

* Migrate to Listr2

* WIP

* Complete engine checks

* Typo, concurrent, simpler output

* Fix broken quit process, add quit install message

* Remove ToDo, update code comments

* Make suggested visual changes

* Collapse Listr
@redwoodjs-bot redwoodjs-bot bot added cherry-pick release:feature This PR introduces a new feature topic/cli topic/crwa create-redwood-app labels Nov 5, 2022
@redwoodjs-bot redwoodjs-bot bot closed this Nov 7, 2022
@jtoar jtoar deleted the cherry-pick-next-d75569e86c018ce458d3c850da0f03297b498572 branch December 8, 2022 21:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release:feature This PR introduces a new feature topic/cli topic/crwa create-redwood-app
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant