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

Turbo: tailwind.config.ts + turbo, not working when updating tailwind config #61607

Closed
tszhong0411 opened this issue Feb 3, 2024 · 2 comments · Fixed by vercel/turborepo#7274 or #61682
Labels
bug Issue was opened via the bug report template. linear: turbopack Confirmed issue that is tracked by the Turbopack team. locked Turbopack Related to Turbopack with Next.js.

Comments

@tszhong0411
Copy link
Contributor

tszhong0411 commented Feb 3, 2024

Link to the code that reproduces this issue

https://github.com/tszhong0411/nextjs-debug/tree/tailwind-ts-bug

To Reproduce

  1. Open codesandbox
  2. Change the color of special class in tailwind.config.ts
  3. The nextjs app doesn't reload

Current vs. Expected behavior

Current:
The app doesn't reload after updating the tailwind configuration.

Expected:
The app will reload after updating the tailwind configuration.

Provide environment information

Operating System:
  Platform: darwin
  Arch: arm64
  Version: Darwin Kernel Version 23.2.0: Wed Nov 15 21:59:33 PST 2023; root:xnu-10002.61.3~2/RELEASE_ARM64_T8112
Binaries:
  Node: 20.10.0
  npm: 10.2.3
  Yarn: 1.22.21
  pnpm: 8.15.1
Relevant Packages:
  next: 14.1.1-canary.27 // Latest available version is detected (14.1.1-canary.27).
  eslint-config-next: N/A
  react: 18.2.0
  react-dom: 18.2.0
  typescript: 5.3.3
Next.js Config:
  output: N/A

Which area(s) are affected? (Select all that apply)

Turbopack (--turbo)

Which stage(s) are affected? (Select all that apply)

next dev (local)

Additional context

The situations for updating:

  • tailwind.config.js without turbo: ✅
  • tailwind.config.ts without turbo: ✅
  • tailwind.config.js with turbo: ✅
  • tailwind.config.ts with turbo: ❌

PACK-2393

@tszhong0411 tszhong0411 added the bug Issue was opened via the bug report template. label Feb 3, 2024
@github-actions github-actions bot added the Turbopack Related to Turbopack with Next.js. label Feb 3, 2024
@ForsakenHarmony ForsakenHarmony added the linear: turbopack Confirmed issue that is tracked by the Turbopack team. label Feb 5, 2024
ForsakenHarmony added a commit to vercel/turborepo that referenced this issue Feb 5, 2024
### Description

Just needed to be added to the array so turbopack knows that it needs to
watch it.

Closes PACK-2393
Fixes vercel/next.js#61607
@ForsakenHarmony
Copy link
Contributor

Will be fixed in the next canary after #61682 is merged.

ForsakenHarmony added a commit that referenced this issue Feb 6, 2024
### Turbopack Updates

* vercel/turborepo#7240 <!-- OJ Kwon -
refactor(turbopack): remove deprecated options -->
* vercel/turborepo#7241 <!-- Will Binns-Smith -
Turbopack: Fix Next.js in cross-bundler benchmarks -->
* vercel/turborepo#7274 <!-- Leah - fix(turbopack):
support reloading typescript tailwind config -->

Fixes #61607
Copy link
Contributor

This closed issue has been automatically locked because it had no new activity for 2 weeks. If you are running into a similar issue, please create a new issue with the steps to reproduce. Thank you.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Feb 20, 2024
ForsakenHarmony added a commit that referenced this issue Jul 25, 2024
…turborepo#7274)

### Description

Just needed to be added to the array so turbopack knows that it needs to
watch it.

Closes PACK-2393
Fixes #61607
ForsakenHarmony added a commit that referenced this issue Jul 29, 2024
…turborepo#7274)

### Description

Just needed to be added to the array so turbopack knows that it needs to
watch it.

Closes PACK-2393
Fixes #61607
ForsakenHarmony added a commit that referenced this issue Jul 29, 2024
…turborepo#7274)

### Description

Just needed to be added to the array so turbopack knows that it needs to
watch it.

Closes PACK-2393
Fixes #61607
ForsakenHarmony added a commit that referenced this issue Aug 1, 2024
…turborepo#7274)

### Description

Just needed to be added to the array so turbopack knows that it needs to
watch it.

Closes PACK-2393
Fixes #61607
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Issue was opened via the bug report template. linear: turbopack Confirmed issue that is tracked by the Turbopack team. locked Turbopack Related to Turbopack with Next.js.
Projects
None yet
2 participants