GeezerVisonThe World Wide Web, restored to 800 pixels of glory. | ||||
| Home|About|Gaming cabinet|Archives|How it works|Starter sites | ||||
GeezerVison is showing github.com — [WIP] Open bigger set of configured projects when opening composite project for operations that operate over multiple projects like rename by sheetalkamat · Pull Request #28261 · microsoft/TypeScript Conversation
|
|
Re-posting my comment #30823 (comment): Personally, I will happily trade off performance for "Find all references" and "Rename symbol" that actually works. In https://github.com/strongloop/loopback-next, we have configured TypeScript to treat the entire monorepo as a single project, so that we can use "Find all references" and "Rename symbol". This setup works pretty well for us and I am not aware of any performance or memory limitation when working in VS Code. If the performance of #28261 is similar, then I would happily accept it. Why we want to migrate from our single-project setup to project references when VS Code experience is good now? The problem is that our build times are super slow. We are building each monorepo package independently, because each package has its own Anyhow. Is there a way how to finish #28261 sooner, possibly without performance optimizations and perhaps behind a new feature flag? |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
|
Miroslav Bajtoš (@bajtos) Thank you for your feedback. I am currently investigating #32028 and looking to improve project references editing experience and you might be interested to give that a try as well. |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
|
Thanks, #32028 looks very promising! I'll give it a try and let you know about my experience. |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
|
Closing in favor of #33287 |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
There was an error while loading. Please reload this page.
Successfully merging this pull request may close these issues.
There was an error while loading. Please reload this page.