Merge branch 'master' into Blendify/npm-split-deps

This is correct usage  and allows developers to easily see
what affets the built package and what affects the devOps
pipeline.

bin/preinstall.js

@@ -4,7 +4,7 @@ const package = require('../package.json');
 
 // Sorry everyone, this is the closest we can get to commenting on package.json
 // dependencies :(
-if (package.devDependencies['bourbon-neat'] !== '~1.9') {
+if (package.dependencies['bourbon-neat'] !== '~1.9') {
     // Wyrm is not compatible with Neat 2.0+, and Neat 1.9 at least pins a
     // node-sass version that doesn't require Python 2. The changes to Wyrm to
     // support Neat 2.0+ are all fairly minor changes, but it deeply affects the

package.json

@@ -17,22 +17,24 @@
     "url": "https://github.com/readthedocs/sphinx_rtd_theme/issues"
   },
   "homepage": "https://github.com/readthedocs/sphinx_rtd_theme",
-  "dependencies": {},
-  "devDependencies": {
+  "dependencies": {
     "bourbon": "~4.3",
     "bourbon-neat": "~1.9",
+    "font-awesome": "^4.7.0",
+    "html5shiv": "^3.7.3",
+    "jquery": "^3.4.1",
+    "lato-font": "^3.0.0",
+    "roboto-fontface": "^0.10.0",
+    "wyrm": "^1.0.9"
+  },
+  "devDependencies": {
     "copy-webpack-plugin": "^5.1.1",
     "css-loader": "^3.1.0",
     "file-loader": "^3.0.1",
-    "font-awesome": "^4.7.0",
-    "html5shiv": "^3.7.3",
     "imports-loader": "^0.8.0",
-    "jquery": "^3.4.1",
-    "lato-font": "^3.0.0",
     "mini-css-extract-plugin": "^0.6.0",
     "node-sass": "^4.13.1",
     "optimize-css-assets-webpack-plugin": "^5.0.3",
-    "roboto-fontface": "^0.10.0",
     "sass-loader": "^7.3.0",
     "style-loader": "^0.23.1",
     "webpack": "^4.30.0",
@@ -40,7 +42,6 @@
     "webpack-dev-server": "^3.3.1",
     "webpack-merge": "^4.2.1",
     "webpack-shell-plugin": "^0.5.0",
-    "webpack-watch-files-plugin": "^1.0.3",
-    "wyrm": "^1.0.9"
+    "webpack-watch-files-plugin": "^1.0.3"
   }
 }