-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
Ignore file watcher events for paths in files.exclude #11455
Labels
fixed
Check the Milestone for the release in which the fix is or will be available.
Language Service
performance
Projects
Milestone
Comments
I agree with you. I realized of something similar. On the other hand, I'm stuck with a feature. Do you mind taking a look at #11609 please? Maybe there is a solution that i don't know. Regards. |
please |
bobbrow 已将这个从 Pull Request 移动到 1.21 中的 Done |
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Labels
fixed
Check the Milestone for the release in which the fix is or will be available.
Language Service
performance
Environment
n/a
Bug Summary and Steps to Reproduce
DefaultClient.registerFileWatcher
registers a watcher for the workspace and we send file create/delete/change events to cpptools when they happen.However, I'm seeing that we receive events for changes to the
.git
folder which is in the defaultfiles.exclude
list. This creates a bit of noise for the extension. We should update the file watcher to discard events for files that we don't care about.files.exclude
should be honored and I thinkC_Cpp.files.exclude
should too. I am aware that there isfiles.watcherExclude
, but by default it has a different pattern thanfiles.exclude
for some reason and doesn't cover the entire.git
folder.Configuration and Logs
Other Extensions
No response
Additional context
No response
The text was updated successfully, but these errors were encountered: