-
-
Notifications
You must be signed in to change notification settings - Fork 807
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
Add include_file
option
#337
Conversation
Useful option to specify individual files to watch. I renamed the `watchDir` function to `watchPath` because it can also handle single files fine (`filepath.Walk` emits a single file if passed a file path). Fixes: air-verse#307
Codecov Report
@@ Coverage Diff @@
## master #337 +/- ##
==========================================
+ Coverage 69.16% 70.62% +1.45%
==========================================
Files 7 7
Lines 960 977 +17
==========================================
+ Hits 664 690 +26
+ Misses 225 217 -8
+ Partials 71 70 -1
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
Will see to adding a test as well. |
daab716
to
83ac079
Compare
383fd6f
to
ecb5bac
Compare
Good to go now. This PR is also available for use via |
There is some flakiness on CI related to test |
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.
LGTM totally
Useful option to specify individual files to watch. I renamed the
watchDir
function towatchPath
because it can also handle single files fine (filepath.Walk
emits a single file if passed a file path).Fixes: #307
Example of it working with a pure
include_dir
andinclude_file
config: