Skip to content

Conversation

@lazlojuly
Copy link

This change fixes #5

Note: When running tests, I have noticed that watch extension's tests are skipped. Is there a reason for that?

@coveralls
Copy link

Coverage Status

Coverage remained the same at 93.443% when pulling 0302bfa on lazlojuly:watch-all-files into 3fef559 on codename-:master.

watch file unless file is process.argv[1]

fileTypeIsToWatch = (file) ->
return extensions[0] is '.*' || (~ extensions.indexOf path.extname file)
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What if .* is not first in the list?

@setthase
Copy link
Owner

setthase commented Feb 3, 2017

Tests are skipped because some issues I don't remember right now 😉 If you would like to return them and eventually fix that issue - I would be happy with that.

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.

Can we add file pattern matching to watch extension [reminder to self]

3 participants