Skip to content
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.

Sign up for GitHub

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

Jump to bottom

files.watcherExclude needs explanation of values #29594

Closed
joaomoreno opened this issue Jun 27, 2017 · 3 comments
Closed

files.watcherExclude needs explanation of values #29594

joaomoreno opened this issue Jun 27, 2017 · 3 comments
Assignees
@bpasero
Labels
file-watcher File watcher
Milestone
June 2017

Comments

@joaomoreno
Copy link
Member

Testing #29478

I couldn't get the files.watcherExclude settings to work unless I prefixed by globs with **/. None of these work as expected:

  • README.md
  • src/README.md
  • src/**
@bpasero
Copy link
Member

bpasero commented Jun 27, 2017

Yeah, glob patterns have to be absolute to work.

@bpasero bpasero added this to the June 2017 milestone Jun 27, 2017
@joaomoreno
Copy link
Member Author

You mean they match against the absolute file paths?

@bpasero
Copy link
Member

bpasero commented Jun 27, 2017

Yeah, the only ones that work relative are files.exclude and search.exclude. To be honest, watcher exclude is a bit of a hack and should not be needed with a good file watcher.

@bpasero bpasero added the file-watcher File watcher label Jun 27, 2017
@vscodebot vscodebot bot locked and limited conversation to collaborators Nov 17, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@bpasero bpasero

Labels
file-watcher File watcher
Projects
None yet
Milestone
June 2017
Development

No branches or pull requests
2 participants
@joaomoreno @bpasero