Skip to content

Update to support sass-loader 8.x - #2223

Merged
JeffreyWay merged 3 commits into
laravel-mix:masterfrom
dshoreman:update-sass-loader
Sep 16, 2019
Merged

JeffreyWay merged 3 commits into
laravel-mix:masterfrom
dshoreman:update-sass-loader

Conversation

@dshoreman

@dshoreman dshoreman commented Sep 15, 2019 •

Copy link
Copy Markdown
Contributor

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.
@Nothing-Works

Copy link
Copy Markdown

@dshoreman any idea why this is still failing?

@dshoreman

dshoreman commented Sep 16, 2019 •

Copy link
Copy Markdown
Contributor Author

@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
Copy Markdown

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

@DannyFeliz

Copy link
Copy Markdown

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

5 participants