Remove another dynamic require and add similar deprecation notice

DABH · DABH · commit 2894751d40e6 · 2018-05-20T15:08:12.000-07:00
diff --git a/lib/extendStringPrototype.js b/lib/extendStringPrototype.js
@@ -91,14 +91,14 @@ module['exports'] = function() {
 
   colors.setTheme = function(theme) {
     if (typeof theme === 'string') {
-      try {
-        colors.themes[theme] = require(theme);
-        applyTheme(colors.themes[theme]);
-        return colors.themes[theme];
-      } catch (err) {
-        console.log(err);
-        return err;
-      }
+      console.log('colors.setTheme now only accepts an object, not a string. ' +
+        'If you are trying to set a theme from a file, it is now your (the ' +
+        'caller\'s) responsibility to require the file.  The old syntax ' +
+        'looked like colors.setTheme(__dirname + ' +
+        '\'/../themes/generic-logging.js\'); The new syntax looks like '+
+        'colors.setTheme(require(__dirname + ' +
+        '\'/../themes/generic-logging.js\'));');
+       return;
     } else {
       applyTheme(theme);
     }
diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -1,7 +1,7 @@
 {
     "name": "colors",
     "description": "get colors in your node.js console",
-    "version": "1.2.5",
+    "version": "1.3.0",
     "author": "Marak Squires",
     "homepage": "https://github.com/Marak/colors.js",
     "bugs": "https://github.com/Marak/colors.js/issues",

Original file line number	Diff line number	Diff line change
`@@ -1,7 +1,7 @@`
`1`	`1`	`{`
`2`	`2`	`"name": "colors",`
`3`	`3`	`"description": "get colors in your node.js console",`
`4`		`- "version": "1.2.5",`
	`4`	`+ "version": "1.3.0",`
`5`	`5`	`"author": "Marak Squires",`
`6`	`6`	`"homepage": "https://github.com/Marak/colors.js",`
`7`	`7`	`"bugs": "https://github.com/Marak/colors.js/issues",`