Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

🐛 helpful messaging at end of create-single-spa execution never prints #386

Open
eckdanny opened this issue Aug 12, 2023 · 0 comments · May be fixed by #387
Open

🐛 helpful messaging at end of create-single-spa execution never prints #386

eckdanny opened this issue Aug 12, 2023 · 0 comments · May be fixed by #387

Comments

@eckdanny
Copy link

Having seen the SingleSpa*Generator#finished() src, pointing out I've never seen this actually print to stdout.

Steps to Reproduce

pnpx create-single-spa@4.1.4 --skipInstall

produces:

? Directory for new project __test-output
? Select type to generate single-spa application / parcel
? Which framework do you want to use? react
? Which package manager do you want to use? pnpm
? Will this project use Typescript? No
? Organization name (can use letters, numbers, dash or underscore) org
? Project name (can use letters, numbers, dash or underscore) proj
Initialized empty Git repository in <some-dir>/.git/

Initialized git repository

   create __test-output/package.json
   create __test-output/jest.config.js
   create __test-output/babel.config.json
   create __test-output/.eslintrc
   create __test-output/.gitignore
   create __test-output/.husky/pre-commit
   create __test-output/.prettierignore
   create __test-output/webpack.config.js
   create __test-output/src/root.component.js
   create __test-output/src/root.component.test.js
   create __test-output/src/org-proj.js

No change to package.json was detected. No package manager install will be executed.
Skipping install command: pnpm install


I'm all done. Just run pnpm install to install the required dependencies.

My yeoman foo is pretty weak, but I'm guessing has to do with the finished method (docs) not mapping to an available priority queue (probably should be end).

In any case, the output messaging probably is really helpful, but I've never seen it 😵

# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants