Add warning message for deprecated addDependencyTo option#251
Conversation
index.js
Outdated
| typeof options.addDependencyTo === "object" && | ||
| typeof options.addDependencyTo.addDependency === "function" | ||
| ) { | ||
| console.warn( |
There was a problem hiding this comment.
@RyanZim addDependencyTo is deprecated in favour of result.messages.dependency, postcss-loader >= v1.0.0 will automatically add your imported files to webpack's file watcher. For more information consider https://github.com/postcss/postcss-import#dependency-message-support
http://api.postcss.org/Result.html#warn additionally or as replacement. postcss-loader will emit a warning about it then.
There was a problem hiding this comment.
But let @ai approve it first 😛
|
Logic looks fine. But @michael-ciniawsky suggested better text. |
|
@ai Will update text; you're fine with |
|
Yeap |
RyanZim
force-pushed
the
addDependencyTo
branch
from
bb981ae to
0def127
Compare
|
@ai Fixed; had to do some pretty bad formatting to keep eslint happy. |
3 participants
@ai Not sure if this is done right.