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.

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

Fix and add tests #89

Closed

Conversation

oconnore
Copy link
Contributor

@oconnore oconnore commented Apr 9, 2014

Sorry about that!

test.ok(true, 'Ended without adding a file.');
watcher.close();
}, 1000);
this.on('any', function(filepath) {
Copy link
Owner

Choose a reason for hiding this comment

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

Should this be this.on('all'?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes, oops.

Eric O'Connor added 2 commits April 9, 2014 13:25
This reverts commit 98001931cc3fb2f0ced70d7271d93c8e586f2d45.
@oconnore
Copy link
Contributor Author

oconnore commented Apr 9, 2014

Want me to squash this?

@shama shama closed this in 6174f38 Apr 9, 2014
@shama
Copy link
Owner

shama commented Apr 9, 2014

No worries. I've got a nice merge flow to do that. :)

Thanks for the test and fixes!

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