Fix peer dependencies

[alexander-akait]

This PR contains a:

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

Motivation / Use-Case

fix peerDeps

Breaking Changes

No

Additional Info

/cc @arcanis is it right way?

Also i still get warning from npm:

npm WARN sass-loader@7.3.1 requires a peer of node-sass@^4.0.0 but none is installed. You must install peer dependencies yourself.

It is bug on npm side? Because i mark this as optional

[arcanis]

Hey @evilebottnawi! It sounds correct to me indeed 👍

Also i still get warning from npm:

npm WARN sass-loader@7.3.1 requires a peer of node-sass@^4.0.0 but none is installed. You must install peer dependencies yourself.

It is bug on npm side? Because i mark this as optional

Yep, they have a server-side bug, npm/cli#224 (comment)

[alexander-akait]

/cc @arcanis https://github.com/webpack-contrib/sass-loader/blob/next/src/getSassOptions.js#L43

fibers also should be in peerDeep right?

[codecov]

Codecov Report

Merging #746 into next will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             next     #746   +/-   ##
=======================================
  Coverage   97.75%   97.75%           
=======================================
  Files          10       10           
  Lines         178      178           
  Branches       63       63           
=======================================
  Hits          174      174           
  Misses          4        4

---

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 96184e1...94c2b45. Read the comment docs.

[arcanis]

fibers also should be in peerDeep right?

Sorry! Yes indeed 👍