-
Notifications
You must be signed in to change notification settings - Fork 48
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
feat(*): add turbo watch mode #1201
Conversation
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Size Change: 0 B Total Size: 92.9 kB ℹ️ View Unchanged
|
CodSpeed Performance ReportMerging #1201 will create unknown performance changesComparing Summary
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #1201 +/- ##
=======================================
Coverage 77.96% 77.96%
=======================================
Files 65 65
Lines 599 599
Branches 123 123
=======================================
Hits 467 467
Misses 123 123
Partials 9 9
|
package.json
Outdated
"ci:sherif": "sherif --ignore-package \"./examples/*\" --ignore-package \"./websites/*\" --ignore-package \"./docs/*\" --ignore-dependency \"@tanstack/react-query\" ", | ||
"ci:test": "turbo run ci:test", | ||
"ci:test:watch": "turbo run ci:test:watch --parallel", | ||
"ci:test:watch": "turbo watch ci:test", | ||
"ci:test:watch:ui": "turbo run ci:test:watch:ui --parallel", |
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.
I named it ci:test:watch:ui because, unlike ci:test:watch which operates in turbo watch mode, this version has a more intuitive UI in the browser.
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.
How about this? double build will be gone (because of dev
depend on prepack
)
package.json
Outdated
"dev": "concurrently \"pnpm dev:build\" \"pnpm dev:test\"", | ||
"dev:build": "turbo watch build --filter=./packages/**", |
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.
"dev": "concurrently \"pnpm dev:build\" \"pnpm dev:test\"", | |
"dev:build": "turbo watch build --filter=./packages/**", | |
"dev": "concurrently \"pnpm dev:prepack\" \"pnpm dev:test\"", | |
"dev:prepack": "turbo watch prepack", |
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.
That's such a great idea I didn't expect a double build, but thanks!
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.
When I use dev:test
, I realize that we don't need dev:prepack
too
@manudeli The reason we use watch in prepack is to ensure that various aspects like build and eslint are properly checked during development. However, if we remove this, it will only perform a one-time check initially. Is this the direction we intended? |
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.
So cool 👍👍👍 Thanks for your digging this together
Let's run just pnpm dev
in root when we developing
I had a misunderstanding. Since prepack is included in dependsOn for ci:test, when running in the development environment, modifying and saving code will cause the process to rerun from prepack through ci:test. This is why dev:prepack became unnecessary. |
# Overview close toss#1098 - add turbo watch mode - turbo watch mode continuously monitors for real-time code changes and automatically re-runs the task when modifications are detected. docs: https://turbo.build/repo/docs/reference/watch <!-- A clear and concise description of what this pr is about. --> ## PR Checklist - [x] I did below actions if need 1. I read the [Contributing Guide](https://github.com/toss/suspensive/blob/main/CONTRIBUTING.md) 2. I added documents and tests. --------- Co-authored-by: Jonghyeon Ko <jonghyeon@toss.im>
Overview
close #1098
docs: https://turbo.build/repo/docs/reference/watch
PR Checklist