-
Notifications
You must be signed in to change notification settings - Fork 63
feat: add options
validation (schema-utils
)
#65
Conversation
Good idea! This is semver major btw as the loader will throw on invalid options now: https://github.com/webpack-contrib/schema-utils/blob/master/src/validateOptions.js#L20 |
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.
What about sourceMapUrlCallback
?
You're right but luckily
It's a |
Fair enough 🙂 |
856bb48
to
bc52b09
Compare
@michael-ciniawsky can you rebase this on master? 😄 Also might be worth adding a test that it throws on an invalid option type. |
ae9f5ec
to
e516db6
Compare
Codecov Report
@@ Coverage Diff @@
## master #65 +/- ##
=========================================
+ Coverage 90.9% 92.3% +1.39%
=========================================
Files 2 2
Lines 11 13 +2
Branches 2 2
=========================================
+ Hits 10 12 +2
Misses 1 1
Continue to review full report at Codecov.
|
Codecov Report
@@ Coverage Diff @@
## master #65 +/- ##
=========================================
+ Coverage 90.9% 92.3% +1.39%
=========================================
Files 2 2
Lines 11 13 +2
Branches 2 2
=========================================
+ Hits 10 12 +2
Misses 1 1
Continue to review full report at Codecov.
|
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
Blocked by #63