Skip to content

Manage own ignoring instead of with chokidar#212

Closed
zertosh wants to merge 3 commits intomasterfrom
internal-ignore
Closed

Manage own ignoring instead of with chokidar#212
zertosh wants to merge 3 commits intomasterfrom
internal-ignore

Conversation

@zertosh
Copy link
Copy Markdown
Member

@zertosh zertosh commented Apr 26, 2015

To avoid creating chokidar instances (#202 et al.)

@zertosh zertosh mentioned this pull request Apr 27, 2015
4 tasks
@es128
Copy link
Copy Markdown
Contributor

es128 commented Apr 28, 2015

Just FYI you could simplify some of your logic here by using multimatch or micromatch with their built-in array handling instead of minimatch

@zertosh
Copy link
Copy Markdown
Member Author

zertosh commented Apr 28, 2015

thanks @es128 - I'm trying to stick to the matching lib that browserify uses (minimatch via glob), but I liked what I saw in micromatch.

@es128
Copy link
Copy Markdown
Contributor

es128 commented Apr 28, 2015

multimatch is a pretty thin wrapper around minimatch that will just handle the arrays for you.

@zertosh
Copy link
Copy Markdown
Member Author

zertosh commented Jun 23, 2015

superseded by #237 and 7cfb8fa

@zertosh zertosh closed this Jun 23, 2015
@zertosh zertosh deleted the internal-ignore branch June 23, 2015 23:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants