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

Re-enable turbo flag in dev script #6808

Closed
mikeesto opened this issue Jun 3, 2024 · 8 comments · Fixed by #6836
Closed

Re-enable turbo flag in dev script #6808

mikeesto opened this issue Jun 3, 2024 · 8 comments · Fixed by #6836
Labels

Comments

@mikeesto
Copy link
Member

mikeesto commented Jun 3, 2024

We had to disable the --turbo flag in the development script due to an error when updating to Nextjs 14.2 (#6803). It would be good to re-enable it once the underlying issue is resolved.

Steps to Reproduce

  1. Enable the --turbo flag in the dev script.
  2. Run the development server.
Error: Expected process result to be a module, but it was ignored

Debug info:
- Execution of get_written_endpoint_with_issues failed
- Execution of <AppEndpoint as Endpoint>::write_to_disk failed
- Execution of Project::emit_all_output_assets failed
- Execution of VersionedContentMap::insert_output_assets failed
- Execution of all_assets_from_entries failed
- Execution of AppEndpointOutput::output_assets failed
- Execution of AppEndpoint::output failed
- Execution of ClientReferenceGraph::entry failed
- Execution of ClientReferenceGraph::new failed
- Execution of primary_referenced_modules failed
- Execution of <ModuleCssAsset as Module>::references failed
- Execution of ModuleCssAsset::module_references failed
- Execution of ModuleCssAsset::classes failed
- Execution of process_css_with_placeholder failed
- Execution of parse_css failed
- Execution of <PostCssTransformedAsset as Asset>::content failed
- Execution of PostCssTransformedAsset::process failed
- Execution of evaluate_webpack_loader failed
- Execution of Completions::completed failed
- Execution of any_content_changed_of_module failed
- Execution of ProcessResult::module failed
- Expected process result to be a module, but it was ignored

Expected Behavior
The development server should run without errors with the --turbo flag enabled.

@mikeesto mikeesto added the bug label Jun 3, 2024
@ovflowd
Copy link
Member

ovflowd commented Jun 4, 2024

Thanks for opening this!

@styfle
Copy link
Member

styfle commented Jun 5, 2024

@ovflowd I believe this will be fixed in vercel/turborepo#8303

@wbinnssmith Do you know when this fix will be available in Next.js?

@styfle
Copy link
Member

styfle commented Jun 5, 2024

This has been fixed in canary, thanks!

https://github.com/vercel/next.js/releases/tag/v15.0.0-canary.13

@ovflowd
Copy link
Member

ovflowd commented Jun 5, 2024

This has been fixed in canary, thanks!

https://github.com/vercel/next.js/releases/tag/v15.0.0-canary.13

Will it be backported to 14x?

@ovflowd
Copy link
Member

ovflowd commented Jun 6, 2024

Because I'm not sure what other sort of breaking changes we need to 👀 for Next.js v15

@styfle
Copy link
Member

styfle commented Jun 6, 2024

I spoke to @ztanner and we are expecting to release a patch to v14 sometime in the next week or two.

@neslob
Copy link

neslob commented Jun 19, 2024

still happing in 14.2.4 (turbo), for me.

@ovflowd
Copy link
Member

ovflowd commented Jun 19, 2024

still happing in 14.2.4 (turbo), for me.

You should report this as a bug on Vercel's turbo repository, not here :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants