Issue 4871 - Start using ESLint for adblockplusui

.eslintrc.json

Index: .eslintrc.json
diff --git a/.eslintrc.json b/.eslintrc.json
new file mode 100644
index 0000000000000000000000000000000000000000..3742233793c9308f1ca006a91ae52c2e0193d07e
--- /dev/null
+++ b/.eslintrc.json
@@ -0,0 +1,13 @@
+{
+  "extends": "eslint-config-eyeo",
+  "root": true,
+  "env": {
+    "browser": true,
+    "webextensions": true
+  },
+  "globals": {
+    "exports": true,
+    "ext": true,
+    "require": true

[Wladimir Palant, 2017/03/09 15:05:06]

The way I see it, exports and require are mostly listed here for
adblockplusInit.js which is actually a proper CommonJS module. I'd suggest
adding .eslintrc.json to the lib/ directory and declare env: { commonjs: true}
there.

As to require, this is also used by messageResponder.js which is a special case
that could run in both environments. I'd declare require as a global explicitly
in this file only to indicate that.

No other files can/should used exports and require so these globals shouldn't be
defined for the entire repository.

[kzar, 2017/03/10 07:28:58]

Done.

+  }
+}