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

Setup turbopack for development #10

Open
JeelRajodiya opened this issue May 11, 2024 · 27 comments · May be fixed by #104
Open

Setup turbopack for development #10

JeelRajodiya opened this issue May 11, 2024 · 27 comments · May be fixed by #104
Assignees
Labels
🐛 Bug Something isn't working ✨ Enhancement New feature or request hacktoberfest This issue is looking for contribution under hacktoberfest event Priority: Low This issue can probably be picked up by anyone looking to contribute to the project. Status: Available No one has claimed responsibility for resolving this issue.
Milestone

Comments

@JeelRajodiya
Copy link
Member

JeelRajodiya commented May 11, 2024

There are some compilation and hydration errors occurring when the command the dev command with was used with --turbo flag. resolve those errors to use --turbo

image

Relevant:
We are trying to do the following from the docs: https://nextjs.org/docs/architecture/turbopack#usage

@JeelRajodiya JeelRajodiya added Priority: Low This issue can probably be picked up by anyone looking to contribute to the project. ✨ Enhancement New feature or request 🐛 Bug Something isn't working labels May 11, 2024
@JeelRajodiya JeelRajodiya added this to the Platform milestone May 11, 2024
@JeelRajodiya JeelRajodiya changed the title Setup turbo for development Setup turbopack for development May 11, 2024
@JeelRajodiya
Copy link
Member Author

@JeelRajodiya JeelRajodiya self-assigned this Jun 2, 2024
@JeelRajodiya JeelRajodiya added Status: Available No one has claimed responsibility for resolving this issue. good first issue Good for newcomers labels Sep 9, 2024
@JeelRajodiya JeelRajodiya removed their assignment Sep 9, 2024
@RonakSurana-2001
Copy link

Is this issue available to solve
Can I work on this issue
Also I want to ask is this repository under hackoctoberfest 2024

@JeelRajodiya
Copy link
Member Author

@RonakSurana-2001 Yes the repo is participating under hacktoberfest. You can work on it and open a PR.

@JeelRajodiya JeelRajodiya added the hacktoberfest This issue is looking for contribution under hacktoberfest event label Oct 3, 2024
@JeelRajodiya
Copy link
Member Author

@RonakSurana-2001 I am not sure, but maybe upgrading next.js version can fix this issue. it is worth a try.

@RonakSurana-2001
Copy link

@JeelRajodiya I added --turbo in package.json dev
Screenshot 2024-10-03 093352
And I am not seeing any Error.
I have also Added
Screenshot 2024-10-03 093523
in next.config.mjs as you suggested. but it is of no use as error is not coming.

https://github.com/hashicorp/next-mdx-remote?tab=readme-ov-file#installation

This may help

I have raised a PR you can see the change. #86

@JeelRajodiya
Copy link
Member Author

Hello @RonakSurana-2001 , please check your PR. I pushed some changes. Now you will be able to see this error.

image

@JeelRajodiya
Copy link
Member Author

I found these two issues somewhat relevant to ours.
vercel/turborepo#6094
vercel/next.js#44653

@RonakSurana-2001
Copy link

Hmm finding the solution

@RonakSurana-2001
Copy link

Think that issue is not with --turbo
I removed Providers from layout.tsx the error was gone

@JeelRajodiya
Copy link
Member Author

please reflect that in your PR

Think that issue is not with --turbo I removed Providers from layout.tsx the error was gone

@RonakSurana-2001
Copy link

RonakSurana-2001 commented Oct 3, 2024

@JeelRajodiya Can you please check #88 PR.

@pavanydg
Copy link
Contributor

@JeelRajodiya is this issue still open?

@RonakSurana-2001 RonakSurana-2001 removed their assignment Oct 28, 2024
@iamridoydey
Copy link

can I work on it?

@JeelRajodiya JeelRajodiya removed the good first issue Good for newcomers label Nov 2, 2024
@JeelRajodiya
Copy link
Member Author

@pavanydg Let's give the issue to @iamridoydey, since he is new.

@JeelRajodiya
Copy link
Member Author

Please go ahead @iamridoydey.
Please note that, the issue might not even have any solution or the it is originated from the library itself. You can research online to figure out the solution of the issue.

@iamridoydey
Copy link

Thanks @JeelRajodiya . I am looking at it. If I stuck then I will ask you for further help . Thanks for the support

@JeelRajodiya
Copy link
Member Author

Thanks @JeelRajodiya . I am looking at it. If I stuck then I will ask you for further help . Thanks for the support

You can also refer to the conversations in this issue. I have shared some of the related resources.

@iamridoydey
Copy link

Thanks bro @JeelRajodiya

@iamridoydey
Copy link

@JeelRajodiya After using
"next-dev": "next dev --turbo",
on the package.json . I got this error

Webpack is configured while Turbopack is not, which may cause problems.
[0] ⚠ See instructions if you need to configure Turbopack:
[0] https://nextjs.org/docs/app/api-reference/next-config-js/turbo

isn't it conflict with each other?

@JeelRajodiya
Copy link
Member Author

@iamridoydey I am not sure either : ) Maybe we need to configure turbopack itself, You can open a PR if you find any solution.

@iamridoydey
Copy link

@JeelRajodiya still some issues. But i am looking at it

@iamridoydey
Copy link

@iamridoydey I am not sure either : ) Maybe we need to configure turbopack itself, You can open a PR if you find any solution.

I am opening a pull request. Check it. It is working fine now.

@iamridoydey iamridoydey linked a pull request Nov 3, 2024 that will close this issue
@JeelRajodiya
Copy link
Member Author

@iamridoydey
You can try searching online to find any similar issues, (see my previous comment)

There might be two possible causes for the issue

  1. Wrong trubopack configuration in our repository
  2. Issue from next.js itself (if it is resolved from their side, it can be resolved by just upgrading next.js)

There might be any other cause as well, which I might be not aware of.

@iamridoydey
Copy link

I upgrade nextjs ro 15.0.2 there turbopack work just fine. But get issues from other dependancy. So many dependancy are not working with current verions of the nextjs. They wants nextjs 14. There was the actual problem

@JeelRajodiya
Copy link
Member Author

@iamridoydey Can you please share a screenshot of the error. Which dependencies are causing the problem?

@iamridoydey
Copy link

@JeelRajodiya I will but I am doing another project. Can you give me some time? Probably in the next day. By the way I am also working on a next js project where I use turbopack not experimental way. many dependancy need to update for this. So we can have a good discussion after I upload all of the issue. Can we?

@JeelRajodiya
Copy link
Member Author

@iamridoydey Yes, you can take your time.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐛 Bug Something isn't working ✨ Enhancement New feature or request hacktoberfest This issue is looking for contribution under hacktoberfest event Priority: Low This issue can probably be picked up by anyone looking to contribute to the project. Status: Available No one has claimed responsibility for resolving this issue.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants