Skip to content

Move frontend dev dependencies out of production dependency list.

username-removed-636429 requested to merge fix-node-dev-deps into master

What does this MR do?

  1. Moves webpack-dev-server and karma-mocha-reporter into devDependencies since they are not needed in production.
  2. Relaxes strict semvers in for other assets in project.json since we now lock them down in yarn.lock.

Are there points in the code the reviewer needs to double check?

After installing node modules with yarn install --pure-lockfile, the output of yarn check should show no errors ("dedupe" warnings are fine -- that is a separate issue).

Why was this MR needed?

This will allow us to upgrade our dependencies more easily in the future and use semver the way it is meant to be used.

see: https://docs.npmjs.com/misc/semver#caret-ranges-123-025-004

Screenshots (if relevant)

n/a

Does this MR meet the acceptance criteria?

What are the relevant issue numbers?

n/a

Merge request reports