summaryrefslogtreecommitdiff
path: root/tools/node_modules/eslint/lib/rules/no-implicit-globals.js
diff options
context:
space:
mode:
Diffstat (limited to 'tools/node_modules/eslint/lib/rules/no-implicit-globals.js')
-rw-r--r--tools/node_modules/eslint/lib/rules/no-implicit-globals.js55
1 files changed, 55 insertions, 0 deletions
diff --git a/tools/node_modules/eslint/lib/rules/no-implicit-globals.js b/tools/node_modules/eslint/lib/rules/no-implicit-globals.js
new file mode 100644
index 0000000000..f0962cdc7a
--- /dev/null
+++ b/tools/node_modules/eslint/lib/rules/no-implicit-globals.js
@@ -0,0 +1,55 @@
+/**
+ * @fileoverview Rule to check for implicit global variables and functions.
+ * @author Joshua Peek
+ */
+
+"use strict";
+
+//------------------------------------------------------------------------------
+// Rule Definition
+//------------------------------------------------------------------------------
+
+module.exports = {
+ meta: {
+ docs: {
+ description: "disallow variable and `function` declarations in the global scope",
+ category: "Best Practices",
+ recommended: false
+ },
+
+ schema: []
+ },
+
+ create(context) {
+ return {
+ Program() {
+ const scope = context.getScope();
+
+ scope.variables.forEach(variable => {
+ if (variable.writeable) {
+ return;
+ }
+
+ variable.defs.forEach(def => {
+ if (def.type === "FunctionName" || (def.type === "Variable" && def.parent.kind === "var")) {
+ context.report({ node: def.node, message: "Implicit global variable, assign as global property instead." });
+ }
+ });
+ });
+
+ scope.implicit.variables.forEach(variable => {
+ const scopeVariable = scope.set.get(variable.name);
+
+ if (scopeVariable && scopeVariable.writeable) {
+ return;
+ }
+
+ variable.defs.forEach(def => {
+ context.report({ node: def.node, message: "Implicit global variable, assign as global property instead." });
+ });
+ });
+ }
+ };
+
+ }
+};