-
Notifications
You must be signed in to change notification settings - Fork 3.4k
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
feat: add --init-type flag #8081
Conversation
It's gonna be so nice when npm stops silently allowing undefined config to be passed to it. |
Is there any particular reason that |
@jonkoops the default in nodejs is commonjs, before this when you initialized it was implicit, now by default it's fixed. |
Understandable, but |
I'll log a separate issue for this, I think this PR was a good starting point, just needs some tweaking to make it more useful for new users, rather than lock them into legacy module systems. |
See #8159 |
I saw a warning for this supported flag, wanted to fix it.