Merge pull request #128 from edx/implement-eslint-plugin-compat

build(eslint): add browser compatibility linting
openedx · Jan 26, 2018 · 1e83713 · 1e83713
2 parents 3ff281e + 6011d37
commit 1e83713
Show file tree

Hide file tree

Showing 3 changed files with 69 additions and 3 deletions.
diff --git a/.eslintrc.json b/.eslintrc.json
@@ -6,7 +6,8 @@
     }
   },
   "rules": {
-    "react/no-array-index-key": "off"
+    "react/no-array-index-key": "off",
+    "compat/compat": "error"
   },
   "settings": {
     "import/resolver": {
@@ -16,8 +17,10 @@
     }
   },
   "env": {
-    "jest": true
+    "jest": true,
+    "browser": true
   },
+  "plugins": ["compat"],
   "overrides": {
     "files": ["*.stories.jsx", "*.test.jsx"],
     "rules": {

diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -63,6 +63,7 @@
     "eslint-config-airbnb": "^16.0.0",
     "eslint-config-edx": "^4.0.1",
     "eslint-import-resolver-webpack": "^0.8.1",
+    "eslint-plugin-compat": "^2.1.0",
     "eslint-plugin-jsx-a11y": "^5.1.0",
     "eslint-plugin-react": "^7.1.0",
     "extract-text-webpack-plugin": "^3.0.1",
@@ -102,5 +103,9 @@
     "transformIgnorePatterns": [
       "/node_modules/(?!lodash-es/.*)"
     ]
-  }
+  },
+  "browserslist": [
+    "last 2 versions",
+    "not ie < 11"
+  ]
 }