add firecloud/no-for, deprecate better/no-fors fp/no-loops

configs/basic.js

@@ -66,7 +66,7 @@ module.exports = {
     'generator-star-spacing': 'error',
     'getter-return': 'error',
     'global-require': 'error',
-    'guard-for-in': 'error',
+    'guard-for-in': 'off',
     'handle-callback-err': 'error',
     'id-blacklist': 'off',
     'id-length': 'off',

configs/better.js

@@ -8,7 +8,7 @@ module.exports = {
     'better/no-deletes': 'off',
     'better/no-exceptions': 'off',
     'better/no-exports': 'off',
-    'better/no-fors': 'error',
+    'better/no-fors': 'off',
     'better/no-function-expressions': 'off',
     'better/no-ifs': 'off',
     'better/no-imports': 'off',

configs/firecloud.js

@@ -7,6 +7,7 @@ module.exports = {
     'firecloud/import-specifier-newline': ['error', {
       allowMultiplePerLine: false
     }],
+    'firecloud/no-for': 'error',
     'firecloud/no-underscore-prefix-exported': 'error',
     'firecloud/order-imports': 'error',
     'firecloud/padding-line-import-multiple': 'error',

configs/fp.js

@@ -9,7 +9,7 @@ module.exports = {
     'fp/no-events': 'error',
     'fp/no-get-set': 'error',
     'fp/no-let': 'off',
-    'fp/no-loops': 'error',
+    'fp/no-loops': 'off',
     'fp/no-mutating-assign': 'error',
     'fp/no-mutating-methods': 'off',
     'fp/no-mutation': 'off',

rules/import-specifier-newline.js

@@ -57,8 +57,7 @@ module.exports = {
           }
         }
 
-        /* eslint-disable better/no-fors, fp/no-loops, no-plusplus */
-        for (let i = 1; i < specifiers.length; i++) {
+        for (let i of _.range(1, specifiers.length)) {
           const lastTokenOfPreviousSpecifier = sourceCode.getLastToken(specifiers[i - 1]);
           const firstTokenOfCurrentSpecifier = sourceCode.getFirstToken(specifiers[i]);
 
@@ -81,7 +80,6 @@ module.exports = {
             });
           }
         }
-        /* eslint-enable better/no-fors, fp/no-loops, no-plusplus */
       }
     };
   }

rules/index.js

@@ -1,6 +1,7 @@
 module.exports.rules = {
   'import-specifier-newline': require('./import-specifier-newline'),
   'no-underscore-prefix-exported': require('./no-underscore-prefix-exported'),
+  'no-for': require('./no-for'),
   'order-imports': require('./order-imports'),
   'padding-line-import-multiple': require('./padding-line-import-multiple'),
   'underscore-prefix-non-exported': require('./underscore-prefix-non-exported')

rules/no-for.js

@@ -0,0 +1,41 @@
+// NOTE modification of https://github.com/jfmengels/eslint-plugin-fp/blob/8757d3c/rules/no-loops.js
+// to allow for for...in, for...of, while and do...while
+
+/* eslint eslint-comments/no-unlimited-disable: off */
+/* eslint-disable */
+
+const create = function (context) {
+  function reportForLoop(node) {
+    context.report({
+      node,
+      message: 'Unallowed use of `for` loop'
+    });
+  }
+
+  function reportWhileLoop(node) {
+    context.report({
+      node,
+      message: 'Unallowed use of `while` loop. Use recursion instead'
+    });
+  }
+
+  return {
+    ForStatement: reportForLoop
+    // ForInStatement: reportForLoop,
+    // ForOfStatement: reportForLoop,
+
+    // WhileStatement: reportWhileLoop,
+    // DoWhileStatement: reportWhileLoop
+  };
+};
+
+module.exports = {
+  create,
+  meta: {
+    docs: {
+      // description: 'Forbid the use of loops.',
+      description: 'Forbid the use of for loops.',
+      recommended: 'error'
+    }
+  }
+};