Move some unused frontend rules to standard/

Author: robertrossmann

standard/best-practices.js

@@ -25,6 +25,11 @@ module.exports = {
     // instead.
     'func-names': 1,
 
+    // Disallow Use of Alert
+    // This rule is aimed at catching debugging code that should be removed and popup UI elements
+    // that should be replaced with less obtrusive, custom UIs.
+    'no-alert': 1,
+
     // Disallow lexical declarations in case/default clauses
     // This rule disallows lexical declarations (let, const, function and class) in case/default
     // clauses. The reason is that the lexical declaration is visible in the entire switch block but

standard/known-errors.js

@@ -97,6 +97,11 @@ module.exports = {
     // character for example.
     'no-irregular-whitespace': 2,
 
+    // Disallow Iterator (__iterator__)
+    // This rule is aimed at preventing errors that may arise from using the __iterator__ property,
+    // which is not implemented in several browsers.
+    'no-iterator': 2,
+
     // Disallow negated left operand of `in` operator
     'no-negated-in-lhs': 2,
 
@@ -112,6 +117,11 @@ module.exports = {
     // This rule aims to eliminate errors due to multiple spaces inside of a regular expression.
     'no-regex-spaces': 2,
 
+    // Disallow Script URLs
+    // Using javascript: URLs is considered by some as a form of eval. Code passed in javascript:
+    // URLs has to be parsed and evaluated by the browser in the same way that eval is processed.
+    'no-script-url': 2,
+
     // Disallow Self Assignment
     // Self assignments have no effect, so probably those are an error due to incomplete
     // refactoring. Those indicate that what you should do is still remaining.

unused.js

@@ -18,11 +18,6 @@ module.exports = {
     // Unused, too restrictive.
     'id-blacklist': 0,
 
-    // Disallow Use of Alert
-    // Unused, front-end specific. Should be moved to appropriate front-end config as soon as it
-    // exists.
-    'no-alert': 0,
-
     // Disallow arrow functions where they could be confused with comparisons
     'no-confusing-arrow': 0,
 
@@ -40,11 +35,6 @@ module.exports = {
     // Unused, front-end specific.
     'no-implicit-globals': 0,
 
-    // Disallow Iterator (__iterator__)
-    // Unused, front-end specific. Should be moved to appropriate front-end config as soon as it
-    // exists.
-    'no-iterator': 0,
-
     // Disallow Multiline Strings
     // Unused, seems handy. This rule should be enabled when pre-ES5 compatibility is required.
     'no-multi-str': 0,
@@ -54,13 +44,6 @@ module.exports = {
     // default values (if ES6's function parameter defaults are not available).
     'no-param-reassign': 0,
 
-    // Disallow Script URLs
-    // Using javascript: URLs is considered by some as a form of eval. Code passed in javascript:
-    // URLs has to be parsed and evaluated by the browser in the same way that eval is processed.
-    // Unused, front-end specific. Should be moved to appropriate front-end config as soon as it
-    // exists.
-    'no-script-url': 0,
-
     // Disallow ++ and --
     // Unused, these operators are quite useful as long as whitespace is used responsibly around
     // them.