Skip to content
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

Update to support sass-loader 8.x #2223

Merged
merged 3 commits into from Sep 16, 2019
Merged

Update to support sass-loader 8.x #2223

merged 3 commits into from Sep 16, 2019

Conversation

dshoreman
Copy link
Contributor

@dshoreman dshoreman commented Sep 15, 2019

Wraps the options for Dart/Node-sass in a sassOptions object and updates versions of certain packages according to requirements for sass-loader 8.0.0.

Fixes #2206

Fixes the ValidationError when building, caused by sass-loader moving
node/dart-sass options to a special `sassOptions` property.
@dshoreman
Copy link
Contributor Author

dshoreman commented Sep 16, 2019

@Nothing-Works After checking the recent commits it seems AppVeyor's been failing since eslint was added 3 commits back in 501a6d1.

The site isn't loading for me at the moment so I can't check the exact error, but IIRC it was complaining about not being able to find 'node'. Looking at the .eslintrc in the above commit it seems to be referring to line 7 ("node": true under the env key), but what it should be I don't know.

tl;dr: AppVeyor isn't even running because eslint config seems broken, but Travis is passing 🎉

@JeffreyWay JeffreyWay merged commit 719edb1 into laravel-mix:master Sep 16, 2019
@HDVinnie
Copy link

When will a Mix update be pushed that includes this fix?

@DannyFeliz
Copy link

When will a Mix update be pushed that includes this fix?

@HDVinnie Done v5.0.0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Compatibility with sass-loader 8.0.0
4 participants