-
Notifications
You must be signed in to change notification settings - Fork 2k
feat: allow Watch Mode with --root-turbo-json
#10687
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
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
--root-turbo-json
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This currently does not catch any changes if the turbo.json
doesn't reside inside of the repository.
We need to either:
- warn users that
turbo.json
outside of the repository will not have changes picked up - start watching outside the repo in the directory that the root
turbo.json
If we want to get this out, I think the former is the better option. The second one might be a little tricky to make sure we don't watch too many files.
Thanks for the review, @chris-olszewski. Should have all those noob mistakes buttoned up and chose to go with:
|
Description
Unblock Watch Mode from being used with
--root-turbo-json
.Testing Instructions
I tested this manually by using
turbo watch lint --root-turbo-json=./some-dir/turbo.json
in acreate-turbo
and changing that file. The tasks would be picked up and re-ran the same as if I omitted the--root-turbo-json
and used a standard path.I also tried it with a JSONC and it worked.