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
--skip-install option in create-next-app #32367
Comments
|
Adding a Here are the appropriate files: (look for https://github.com/vercel/next.js/blob/canary/packages/create-next-app/index.ts |
|
Hey there, I'm trying to do this. skipping the installation is not a problem, but what about populating |
There are a few methods of doing this in my opinion. I'm listing them down:
@balazsorban44, what do you think should be the optimal solution? PS: I was hoping I could work on this, as I have already started too. Am I too late? |
|
Feel free to work on adding the I'm not sure about what @anirudh1713 is suggesting. The CLI is meant to scaffold a minimal Next.js example project. You can use the |
|
When using the default templates from this folder, the dependency list in So what would be a good method to get the respective versions? Should we just mark it as |
|
@Pika-Pool Yes, that is what I was trying to say in my previous comment, |
|
Thank you for the explanation. The CLI is expected to spin up the |
- adds feature issue vercel#32367 - facilitates using any package manager, not just npm/yarn - for default templates, writes "latest" dist tag for deps in package.json
|
I've added a PR(#32484) to add this feature. It adds "latest" as the version for each dependency for the default templates. TestsFor this feature, what all should the tests cover? This is what I think, please add/remove tests as required:
Is it necessary to add tests for Where to add the tests?I am not able to find where to add them. There are tests for testing readme mentions that "integration: These tests run misc checks and modes and is where tests used to be added before we added more specific folders. We should not add any more tests here.". So where should I add the new tests for |
|
Feel free to add it in https://github.com/vercel/next.js/blob/canary/test/integration/create-next-app/index.test.js We will update the README to avoid future confusion. |
As mentioned in #32679 (comment) it looks like these externals need to be updated to the compiled path not that they are ncc'd. Also updated the test readme a bit to reduce confusion in #32367 (comment)
Describe the feature you'd like to request
A
--skip-installoption in create-next-app, like in react-native, to skip installing dependencies would be really great. It allows using other package managers likepnpmDescribe the solution you'd like
do not install dependencies automatically when the
--skip-installoption is set. But the dependencies should be listed inpackage.json, so user could manually install the dependencies using other package managers likepnpmDescribe alternatives you've considered
An option to use
pnpm(--use-pnpm).pnpmhas gained significant popularity and would be a good option to add.The text was updated successfully, but these errors were encountered: