-
-
Notifications
You must be signed in to change notification settings - Fork 67
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
Avoid check old webpack versions #66
Conversation
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.
@alexfedoseev or I can merge once one comment is addressed.
src/loader.js
Outdated
Can't find sass resources in your config. | ||
Make sure ${isModernWebpack ? 'loader.options.resources' : 'webpackConfig.sassResources'} exists. | ||
`); | ||
const error = new Error('Can\'t find sass resources in your config. Make sure loader.options.resources'); |
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.
Missing exists.
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.
Added:
@david-ogalego any update? |
@alexfedoseev are you OK with this one? |
Yeah, totally. |
This reverts commit 91d097e. This allows this commit to go into v2.0.0.
This reverts commit 91d097e. This allows this commit to go into v2.0.0.
If someone wants to use older webpack versions, can download older versions of loader.
This change is