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

feat(cna): add --turbo flag and prompt for enabling Turbopack dev #65925

Closed
wants to merge 5 commits into from

Conversation

devjiwonchoi
Copy link
Member

Closes #65924

@ijjk ijjk added create-next-app Related to our CLI tool for quickly starting a new Next.js application. tests labels May 18, 2024
@ijjk
Copy link
Member

ijjk commented May 18, 2024

Allow CI Workflow Run

  • approve CI run for commit: 380f083

Note: this should only be enabled once the PR is ready to go and can only be enabled by a maintainer

@@ -222,4 +223,32 @@ describe.skip('create-next-app --app (App Router)', () => {
})
})
})

it('should enable turbopack on --turbo flag', async () => {
Copy link
Member Author

@devjiwonchoi devjiwonchoi May 18, 2024

Choose a reason for hiding this comment

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

The test passes, but the describe on top is skipped since #65845

Copy link
Contributor

Choose a reason for hiding this comment

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

hello good work ❤️ , I did a study, I wonder how your test coverage does not get stuck in timeout, I saw that you also got stuck in timeout

test/integration/create-next-app/templates/app.test.ts

@devjiwonchoi devjiwonchoi marked this pull request as ready for review May 18, 2024 14:42
@devjiwonchoi devjiwonchoi requested a review from a team as a code owner May 18, 2024 14:45
@devjiwonchoi devjiwonchoi requested review from ismaelrumzan and jh3y and removed request for a team May 18, 2024 14:45
@ijjk ijjk added the Documentation Related to Next.js' official documentation. label May 18, 2024
@@ -272,6 +279,7 @@ async function run(): Promise<void> {
importAlias: '@/*',
customizeImportAlias: false,
empty: false,
turbo: false,
Copy link
Member Author

Choose a reason for hiding this comment

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

default value false until areweturboyet 100

@devjiwonchoi
Copy link
Member Author

Closing as duplicate of #65926

devjiwonchoi added a commit to Arindam200/next.js that referenced this pull request May 18, 2024
@devjiwonchoi devjiwonchoi deleted the cna-dev-turbo branch May 18, 2024 20:31
@github-actions github-actions bot added the locked label Jun 2, 2024
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jun 2, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
create-next-app Related to our CLI tool for quickly starting a new Next.js application. Documentation Related to Next.js' official documentation. locked tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add option to enable Turbopack with create-next-app
3 participants