这是indexloc提供的服务,不要输入任何密码
Skip to content

fix(watch): allow usage of turbo.jsonc #10340

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

Merged
merged 5 commits into from
Apr 15, 2025
Merged

fix(watch): allow usage of turbo.jsonc #10340

merged 5 commits into from
Apr 15, 2025

Conversation

anthonyshew
Copy link
Contributor

@anthonyshew anthonyshew commented Apr 14, 2025

Description

Fixes #10323

Testing Instructions

I'm testing by hand as I'm aware its notoriously difficult to write tests in this area of the codebase.

@anthonyshew anthonyshew requested a review from a team as a code owner April 14, 2025 22:54
Copy link

vercel bot commented Apr 14, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
examples-basic-web ✅ Ready (Inspect) Visit Preview 💬 Add feedback Apr 15, 2025 2:04pm
examples-designsystem-docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Apr 15, 2025 2:04pm
examples-gatsby-web ✅ Ready (Inspect) Visit Preview 💬 Add feedback Apr 15, 2025 2:04pm
examples-kitchensink-blog ✅ Ready (Inspect) Visit Preview 💬 Add feedback Apr 15, 2025 2:04pm
examples-native-web ✅ Ready (Inspect) Visit Preview 💬 Add feedback Apr 15, 2025 2:04pm
examples-nonmonorepo ✅ Ready (Inspect) Visit Preview 💬 Add feedback Apr 15, 2025 2:04pm
examples-svelte-web ✅ Ready (Inspect) Visit Preview 💬 Add feedback Apr 15, 2025 2:04pm
examples-tailwind-web ✅ Ready (Inspect) Visit Preview 💬 Add feedback Apr 15, 2025 2:04pm
examples-vite-web 🔄 Building (Inspect) Visit Preview 💬 Add feedback Apr 15, 2025 2:04pm
turbo-site ✅ Ready (Inspect) Visit Preview 💬 Add feedback Apr 15, 2025 2:04pm

Copy link
Member

@chris-olszewski chris-olszewski left a comment

Choose a reason for hiding this comment

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

This will avoid the error, but we need to implement the correct behavior so that turbo.jsonc triggers a package watcher event.

@anthonyshew
Copy link
Contributor Author

anthonyshew commented Apr 15, 2025

@chris-olszewski 🤦 When I hand-tested it, I was using a Just-in-Time package, so of course it looked like it worked.

I added some code that for filewatching (I hope) and used a Compiled Package now and hand-tested again. It looks like it works (for real this time) - but I only tested a simple case.

Do I look like I'm more on the right track now? I'm all but certain it's not all the way right, but I'll keep going if you say I'm headed the right direction.

Copy link
Member

@chris-olszewski chris-olszewski left a comment

Choose a reason for hiding this comment

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

LGTM, only thing that really needs to get changed is the change to the exit handling in the package watcher

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

Successfully merging this pull request may close these issues.

Watch mode doesn't work inside a project with nested turbo.jsonc
2 participants