-
Notifications
You must be signed in to change notification settings - Fork 41
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
Trunk+VSCode keeps my coffee warm #287
Comments
Thanks for the feedback @Oni-giri! We are actively working on reducing the resource usage of the vscode extension and will be sure to update this ticket when that work makes its way into a release. |
Thanks for posting the screenshot. That sort of information is definitely helpful to us. Can you answer a few other questions:
We're still actively working on trying to make resource usage less of a problem in general, and we will keep you updated when we have a fix you can try! |
actions:
disabled:
- trunk-announce
- trunk-check-pre-push
- trunk-fmt-pre-commit
enabled:
- trunk-upgrade-available |
Hi @Oni-giri, we have made a lot of changes since 1.5.1 so we recommend you stay up to date if possible. In 1.11.1, we made some optimizations to clean up stale jobs that should rein in the amount of resources that the Trunk extension is using. We also have a forthcoming release of the extension that will allow configuring the extension's CPU usage. If you have any issues upgrading to 1.11.1, let us know! |
Thanks @laurit17 for the answer! I just upgraded, I'll keep you updated. |
Hi @laurit17, new issue: Trunk is eating my memory alive!
I can send you the full report by mail if you want. |
I would really appreciate any info you are able to provide, thanks. feel free to email any info you have to lauri [at] trunk [dot] io. I'm also curious about your usage patterns. We'd expect fairly light memory usage from the trunk binary itself, with spikes from running linters. Curious - do you also see this only when using the vscode extension. Do you run trunk from the command line as well? DO you get a high memory footprint if you are doing that too? |
Please check out the latest version of trunk |
Hi @Oni-giri, have you had the chance to try this out on the latest versions ( |
This is very much still an issue, btw. I have to kill groups of 4-6 Do any of the Trunk team actually use Trunk in a real-world project, with VS Code on a Mac? cli:
version: 1.21.0
plugins:
sources:
- id: trunk
ref: v1.4.5 |
Hi @philsherry, Sorry you are experiencing this. For context, trunk will launch a daemon for every git repository you open. This daemon process will currently stay alive forever, but the next version of trunk will have the daemon auto shutdown after some period of inactivity. Could you please provide more information about what you are seeing?
Thank you for your feedback! |
👋🏻 @det Have you tested this with many projects that use git submodules? Due to the nature of my work, I might have two instances of VS Code running, each with at least one submodule. Trunk CPU usage can be anywhere from ~40% to 90% during these times. The thing is, I might never need to lint any of the files in those submodules—is it still spun up regardless, even if I have that directory in the ignore paths? Could there be some kind of defer option available? “Don’t start trunking on this tree until the user issues a command in a file” kinda thing? (I’m literally thinking as I’m typing, so this might be a garbage idea, or even something that you already do.) |
Hello,
Thanks for the awesome "master-linter" - the UI/UX is great, it's very useful.
Just a little feedback, share by other dev friends : it's heavy on the processor, and we had to uninstall it (at least from VSCode) to solve the issue. After 5 minutes of code, compute usage would shoot to 100%, and my MacBook Air 2021 would get very, very warm.
I think the VSCode plugin is at fault, however the behavior persisted after I removed it, and stopped when I did a clean uninstallation. I reinstalled only the CLI tool after, which solved the problem.
Thanks for the good work, I hope this feedback helps :-)
The text was updated successfully, but these errors were encountered: