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

fix: support node-sass v5.0.0 #899

Merged
merged 2 commits into from Nov 2, 2020
Merged

fix: support node-sass v5.0.0 #899

merged 2 commits into from Nov 2, 2020

Conversation

nhevia
Copy link
Contributor

@nhevia nhevia commented Nov 1, 2020

New projects with node-sass are returning an error due to latest being bumped to v5

Error: Node Sass version 5.0.0 is incompatible with ^4.0.0.

This PR contains a:

  • bugfix
  • new feature
  • code refactor
  • test update
  • typo fix
  • metadata update

Motivation / Use-Case

Every new (or updated) project using node-sass latest version will result in compilation errors during to mismatch semver.

Breaking Changes

Supports >4.0.0, open to changes.

Additional Info

None

@jsf-clabot
Copy link

jsf-clabot commented Nov 1, 2020

CLA assistant check
All committers have signed the CLA.

@nhevia nhevia changed the title Support node-sass v5.0.0 fix: support node-sass v5.0.0 Nov 1, 2020
nhevia and others added 2 commits November 1, 2020 02:47
New projects with node-sass are returning an error due to latest being bumped to v5

`Error: Node Sass version 5.0.0 is incompatible with ^4.0.0.`
@codecov
Copy link

codecov bot commented Nov 1, 2020

Codecov Report

Merging #899 into master will not change coverage.
The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #899   +/-   ##
=======================================
  Coverage   96.01%   96.01%           
=======================================
  Files           4        4           
  Lines         226      226           
  Branches       72       72           
=======================================
  Hits          217      217           
  Misses          8        8           
  Partials        1        1           
Impacted Files Coverage Δ
src/utils.js 95.21% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update ce6b3b8...2579fd8. Read the comment docs.

@nhevia
Copy link
Contributor Author

nhevia commented Nov 1, 2020

Fixes #898

Copy link
Member

@alexander-akait alexander-akait left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks

@alexander-akait alexander-akait merged commit c3e279f into webpack-contrib:master Nov 2, 2020
@nhevia nhevia deleted the patch-1 branch November 2, 2020 15:13
kodiakhq bot pushed a commit to vercel/next.js that referenced this pull request Nov 9, 2020
`node-sass` v5 introduced support for Node.js v15, which is not supported by v4. However, Next.js currently errors with
```
Error: Node Sass version 5.0.0 is incompatible with ^4.0.0.
```
when attempting to build with `node-sass` 5.0.0. This error comes from `sass-loader`. They have recently released version 10.0.5 which supports `node-sass` 5.0.0 (PR <webpack-contrib/sass-loader#899>, release <https://github.com/webpack-contrib/sass-loader/releases/tag/v10.0.5>).
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.

None yet

4 participants