-
Notifications
You must be signed in to change notification settings - Fork 213
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
allow a single build/watch command for an entire workspace #3804
Comments
Sounds reasonable, thanks. I guess with renewed interest in |
looking forward to seeing what you guys do with it! |
I just opened #3806 which maybe overlaps a little. But my idea wasn't to restrict it to a single Pub workspace, but rather have a single instance an IDE could spawn that could handle all projects in the open set of folders (which may or may not be a workspace). Probably there are some gains to having only a single build_runner instance (even if it's just spawning multiple isolates) versus spawning many build_runner instances for the different projects in the (VS Code) workspace. |
See also my comment regarding an analyzer plugin, this would also solve the workspace issue. The plugin works on analysis contexts, you will only get one context for the whole workspace, and anything open in your IDE which has an analysis context will get codegen run on it. |
I had a similar idea a while ago: #2981. At least with the standalone |
Right, build_runner has quite baked in the notion of a single "root" package. Workspaces do also technically have a "root" package - (at the workspace root) - but we would need to be able to allow other packages to also behave like root packages in terms of running their build-to-source builders and such. It would definitely require some thoughtful design work. |
right now I spawn 3 or 4 different build_runner watches when I open a project, but with the advent of workspaces it feels like that shouldn't be necessary any more.
Is it possible for one build_runner to cover all of the projects in a workspace?
The text was updated successfully, but these errors were encountered: