Skip to content

[RFC 638] Interactive way to create new Ember apps and addons#9824

Merged
bertdeblock merged 1 commit into
ember-cli:masterfrom
bertdeblock:rfc-638-interactive-new
Sep 30, 2022
Merged

[RFC 638] Interactive way to create new Ember apps and addons#9824
bertdeblock merged 1 commit into
ember-cli:masterfrom
bertdeblock:rfc-638-interactive-new

Conversation

@bertdeblock

@bertdeblock bertdeblock commented Mar 16, 2022

Copy link
Copy Markdown
Member

RFC PR: emberjs/rfcs#638

Went with inquirer for the following reasons:

  • Development of enquirer seems to have stalled
  • prompts seems nice as well, but inquirer is more populair (based on download and GH star count)
  • inquirer seems to be the most active library at the moment

Result:

Screen.Recording.2022-09-30.at.16.04.50.mov

@bertdeblock

Copy link
Copy Markdown
Member Author

I overlooked some tests, e.g. ember help output, will fix asap!

@bertdeblock bertdeblock force-pushed the rfc-638-interactive-new branch from 74ee8f9 to 846b285 Compare March 16, 2022 13:48
@NullVoxPopuli

Copy link
Copy Markdown
Contributor

excited to try this out!

@bertdeblock

Copy link
Copy Markdown
Member Author

Can't seem to figure out why CI is failing at the moment.
It seems this line is the culprit:
https://github.com/ember-cli/ember-cli/blob/master/tests/acceptance/smoke-test-slow.js#L236-L237
It hangs, and thus the timeout is exceeded.
Seems to fail here as well:
https://github.com/ember-cli/ember-cli/runs/5564343698?check_suite_focus=true#step:5:2237

Comment thread package.json
@bertdeblock bertdeblock force-pushed the rfc-638-interactive-new branch from 846b285 to c2d8688 Compare April 12, 2022 18:02
@NullVoxPopuli

Copy link
Copy Markdown
Contributor

Really excited for this!!! An interactive mode is so clutch for folks new to programming and are extremely intimidated by help screens and cli flags.

(It's also just a nice way to walk folks through stuff)

@bertdeblock

Copy link
Copy Markdown
Member Author

Glad you're excited 😄 I just need to know what to do about #9824 (comment).

@rwjblue rwjblue left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Overall this looks great, thanks for working on it!!

I left a couple inline questions/suggestions, and I think we need at least one test that goes through a full flow "acceptance" / "slow" test that actually runs ember new and enters the values for the prompts (to make sure that things actually work in reality).

Comment thread lib/tasks/interactive-new.js Outdated
Comment thread lib/tasks/interactive-new.js Outdated
@bertdeblock bertdeblock force-pushed the rfc-638-interactive-new branch from 59dc359 to b6f28f8 Compare April 14, 2022 11:56
@bertdeblock

bertdeblock commented Apr 14, 2022

Copy link
Copy Markdown
Member Author

Addressed the smaller issues, still need to come up with an acceptance test that completes the entire flow.

@locks locks force-pushed the rfc-638-interactive-new branch from b6f28f8 to 2c2aa89 Compare April 19, 2022 23:57
@bertdeblock bertdeblock self-assigned this Jun 3, 2022
@bertdeblock

Copy link
Copy Markdown
Member Author

In case anyone is following along, I could use some help writing a full acceptance test where we simulate user input.

@kategengler

Copy link
Copy Markdown
Member

Discussed in the ember-cli meeting today, we've decided not to block on the interactive testing but hope to add that test coverage later.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants