-
Notifications
You must be signed in to change notification settings - Fork 51
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
Ignored files shouldn't appear in the Watching
output
#27
Comments
Do you have an example of the command and it's output? |
Try it on here: https://github.com/mhkeller/chokidar-ignore |
Ok thanks. chokidar-cli is just logging all "patterns" which have been given to it for watching. In this case it seems like your terminal is doing an expansion for the pattern before chokidar-cli receives it and actually this command will be run: |
Ya that's what I figured L170-176 was doing. For now maybe a note in the readme would help? I spent a bit of time thinking my path was wrong before realizing it was actually ignoring as designed. |
Chokidar has a I'd suggest the confusion could be avoided by just expanding the message to echo back both user inputs: console.error('Watching', '"' + watchlist + '" and ignoring "' + ignorelist + '" ..'); |
Ya that would certainly help. |
Chokidar properly ignores files passed in through
-i
but the console text will still say it is watching these files. This appears to be the relevant code: https://github.com/kimmobrunfeldt/chokidar-cli/blob/89e1310d265c4d9dfe38ff0cc5458bdf3c36b32c/index.js#L170-L176Edit by @kimmobrunfeldt: Made the link to explicitly state a commit instead of master branch.
The text was updated successfully, but these errors were encountered: