fix: engine warning on console for ui

[juanpicado]

Type: bug

The following has been addressed in the PR:

• There is a related issue?
• Unit or Functional tests are included in the PR

Description:

"engines": {
    "node": ">= 10.13.0",
    "node": ">= 8",
    "npm": ">=5"
  },

We still support v8 and this is a dependency of verdaccio.

Resolves https://github.com/verdaccio/ui/issues/402

[codecov]

Codecov Report

❗ No coverage uploaded for pull request base (master@787dda4). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #403   +/-   ##
=========================================
  Coverage          ?   87.81%           
=========================================
  Files             ?      137           
  Lines             ?      952           
  Branches          ?      211           
=========================================
  Hits              ?      836           
  Misses            ?      102           
  Partials          ?       14

[juanpicado]

Yes, it does, but I think nobody is developing on Node 8` anyway. The change webpack-contrib/url-loader#198 does not affect the library behavior.

This is just a dependency and we ship a webpack bundle. I updated the README.

[DanielRuf]

This is just a dependency and we ship a webpack bundle. I updated the README.

You are right. I saw the other comment about the devDependency so this is not relevant for the prod bundles as you wrote =)

[verdacciobot]

Thanks for your PR, the @verdaccio/ui package will be accessible from here for testing purposes:

npm install @verdaccio/ui-theme@v0.3.10-cdafa78-pr403.0 --registry https://registry.verdaccio.org