-
Notifications
You must be signed in to change notification settings - Fork 4
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
Pass along gaze watch parameters so that there is more control over file watching. #6
Conversation
Suggested new version is minor change, non-breaking.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great job, thank you! Just a few minor NITs
index.js
Outdated
@@ -16,7 +17,8 @@ function startWatching({ watch, match, exec }, { verbose }) { | |||
return new Promise(resolve => { | |||
match = match || [ '**/*' ]; | |||
const prefix = [ list('watch', watch), list('match', match), list('exec', exec) ].join(' '); | |||
const gaze = new Gaze(watch); | |||
console.log(JSON.stringify(options.gazeOptions)); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could you please use log()
function instead of console.log()
? I would like to keep ability to have it completely silent.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
will do.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I will remove this completely.
package.json
Outdated
@@ -1,6 +1,6 @@ | |||
{ | |||
"name": "run-when-changed", | |||
"version": "2.0.2", | |||
"version": "2.1.0", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I can rev the version and create a git tag at the same time when I publish.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
okay, I will move this back..
Okay, I have updated the code based on your suggestions.
|
Should this pull request be merged? The asked for changes have been made. Cheers! |
The use case is described in issue #5