fix: peer deps

[alexander-akait]

This PR contains a:

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

Motivation / Use-Case

fix peer deps

Breaking Changes

No

Additional Info

/cc @arcanis

need your advice in this situation, by default we use file-loader if we can't decode content to base64 (limit option), but we allow to setup custom fallback (https://github.com/webpack-contrib/url-loader#fallback, allow to use any other loader), so we can't setup right peer deps in this case. What we should do?

[codecov]

Codecov Report

Merging #189 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff          @@
##           master   #189   +/-   ##
=====================================
  Coverage     100%   100%           
=====================================
  Files           3      3           
  Lines          38     38           
  Branches       10     10           
=====================================
  Hits           38     38

---

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 1b9dbd1...21f627a. Read the comment docs.

[arcanis]

What we should do?

I would recommend to slightly change the documentation to recommend your users to use require.resolve. This will ensure that the resolution is done from whichever file declares the fallback rather than from the point of view of url-loader:

module.exports = {
  module: {
    rules: [
      {
        test: /\.(png|jpg|gif)$/i,
        use: [
          {
            loader: 'url-loader',
            options: {
              fallback: require.resolve('responsive-loader'),
            },
          },
        ],
      },
    ],
  },
};

[alexander-akait]

@arcanis thanks for reply

[alexander-akait]

Docs updated, new PR for support pnp #195