Don't add a watcher to files which should be ignored.

[codeviking]

If you call chokidar.watch(path, { ignored: '/some/string/that/matches/path' }); it'll still add a watcher to the file.

This adds a filter in watchFile which prevents the watcher from getting added if it matches the ignored glob.

This avoided watching files in node_modules for a particular project I'm working on and also corrected a segfault which would occur as a result (because of the watch handlers attached to files which were removed via an npm update call).

[zertosh]

Thanks @codeviking! I wrote this up already in https://github.com/substack/watchify/pull/212 but forgot about it (prob because it meant bumping the major version). A few things:

• I want to switch away from anymatch to minimatch so globing is inline with browserify (bugs and all). But I'll take anymatch for now since that means I don't need to bump the major version.
• The result of the glob test should be cached.
• Spaces in the file should be left alone - and functions should exit early if possible (also minimizes the diff)
• Commits should be squashed.

If you want to make these changes, then I can look at your PR again - otherwise, I can move https://github.com/substack/watchify/pull/212 to anymatch and merge that tmw.

[codeviking]

I'm actually traveling for a week but I'm happy to make the changes once I return. No rush in merging as we're just using my fork in the interim.

Thanks!

[codeviking]

@zertosh Checking back in. I'll make these changes this morning and submit you an updated PR.