Move mongodb and redis from optionalDependencies to devDependencies

tonila · Feb 12, 2021 · 487691c · 487691c
1 parent 5bee2dc
commit 487691c
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 6 deletions.
diff --git a/README.md b/README.md
@@ -34,9 +34,9 @@ new ACL.redisBackend({ redis, prefix = "acl_" })
 
 * The `mongodb` dependency upgraded from v2 to the latest v3.
 
-* Both `mongodb` and `redis`  dependencies moved to `optionalDependencies`
+* Both `mongodb` and `redis` dependencies moved to `devDependencies`. You have to install them to your project separately.
 
-* The minimal supported nodejs version was `0.10`, but became the current LTS `10`
+* The minimal supported nodejs version was `0.10`, but became the current LTS `10`. But, `acl2` runs fine on node `8`.
 
 * The first published version of `acl2` is `1.0` to be more SemVer compliant.
 
@@ -71,6 +71,14 @@ Using npm:
 npm install acl2
 ```
 
+Optionally: 
+
+```shell script
+npm install mongodb@3
+
+npm install redis@2
+```
+
 ## Documentation
 
 - [addUserRoles](#addUserRoles)

diff --git a/package.json b/package.json
@@ -25,11 +25,9 @@
     "chai": "^4.2.0",
     "eslint": "^7.7.0",
     "mocha": "^8.1.1",
-    "nyc": "^15.1.0",
-    "prettier": "^2.1.0"
-  },
-  "optionalDependencies": {
     "mongodb": "^3.0.0",
+    "nyc": "^15.1.0",
+    "prettier": "^2.1.0",
     "redis": "^2.2.5"
   },
   "scripts": {