Merge pull request #32 from Brightspace/s-akulick/no-console-web-refa…

…ctor

Refactored web configs and added no-console rule

browser-config.js

@@ -1,6 +1,21 @@
+const { sortMemberRules } = require('./sort-member-config');
+
 module.exports = {
-  "extends": "./index.js",
-  "env": {
-    "browser": true
-  }
+	"extends": "./index.js",
+	"parser": "babel-eslint",
+	"env": {
+		"browser": true
+	},
+	"plugins": [
+		"html",
+		"sort-class-members"
+	],
+	"globals": {
+		"D2L": false,
+		"Promise": false
+	},
+	"rules": {
+    "no-console": ["error", { "allow": ["warn", "error"] }],
+		...sortMemberRules
+	}
 };

lit-config.js

@@ -1,21 +1,11 @@
-const { sortMemberRules } = require('./sort-member-config');
-
 module.exports = {
-	"extends": "./index.js",
-	"parser": "babel-eslint",
+	"extends": "./browser-config.js",
 	"env": {
-		"browser": true,
 		"es6": true
 	},
 	"plugins": [
-		"lit",
-		"html",
-		"sort-class-members"
+		"lit"
 	],
-	"globals": {
-		"D2L": false,
-		"Promise": false
-	},
 	"rules": {
 		"arrow-spacing": 2,
 		"no-confusing-arrow": 2,
@@ -41,7 +31,6 @@ module.exports = {
 		"lit/no-template-bind": 2,
 		"lit/no-template-map": 0,
 		"lit/no-useless-template-literals": 2,
-		"lit/no-value-attribute": 2,
-		...sortMemberRules
+		"lit/no-value-attribute": 2
 	}
 };

package.json

@@ -1,6 +1,6 @@
 {
   "name": "eslint-config-brightspace",
-  "version": "0.6.7",
+  "version": "0.7.0",
   "description": "Common Brightspace eslint configs.",
   "main": "index.js",
   "scripts": {

polymer-3-config.js

@@ -1,21 +1,6 @@
-const { sortMemberRules } = require('./sort-member-config');
-
 module.exports = {
-	"extends": "./index.js",
-	"parser": "babel-eslint",
-	"env": {
-		"browser": true
-	},
-	"plugins": [
-		"html",
-		"sort-class-members"
-	],
-	"globals": {
-		"D2L": false,
-		"Polymer": false
-	},
+	"extends": "./browser-config.js",
 	"rules": {
-		"strict": 0,
-		...sortMemberRules
+		"strict": 0
 	}
 };

polymer-config.js

@@ -1,15 +1,5 @@
 module.exports = {
-  "extends": "./index.js",
-  "env": {
-    "browser": true
-  },
-  "plugins": [
-    "html"
-  ],
-  "globals": {
-    "D2L": false,
-    "Polymer": false
-  },
+  "extends": "./browser-config.js",
   "rules": {
     "strict": 0
   }