-
-
Notifications
You must be signed in to change notification settings - Fork 29
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
feat: add support for contextDependencies
in the {Object}
interface (options.contextDependencies
)
#23
Conversation
Codecov Report
@@ Coverage Diff @@
## master #23 +/- ##
==========================================
+ Coverage 96.15% 96.42% +0.27%
==========================================
Files 2 2
Lines 26 28 +2
Branches 7 8 +1
==========================================
+ Hits 25 27 +2
Misses 1 1
Continue to review full report at Codecov.
|
contextDependencies
in the {Object}
interface (options.contextDependencies
)
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.
@raleksandar Thx
@jhnns Friendly ping to raise concerns @d3viant0ne @evilebottnawi Could someone please merge this within the next ~48 hours or so, I haven't write access here :) |
@michael-ciniawsky id don't have access as well |
@michael-ciniawsky - Published & available in Access issues should also be resolved for both of you, the push access config was missing a team. |
Awesome, thanks guys! |
This change adds support for declaring directories as dependencies by using
contextDependencies
in object interface returned by the generator function.Background: I've been using val-loader to generate modules based on some configuration files which are grouped in a single directory. In my generator function, I traverse that directory and parse all found configuration files adding them to the dependencies array. Problem is that in watch mode if a new file is added in that directory it will not be picked up by the webpack and I need to stop it and rerun it again.
Regarding the tests, I have simply added
contextDependencies
totest/fixtures/depenndencies.js
in order to make codecov happy :)If I need to do something more please let me know.
Hopefully, you will find this addition welcome :)