From 5f3bad533843ca939e46bbec05ea358c32d28fa7 Mon Sep 17 00:00:00 2001 From: JounQin Date: Thu, 21 Oct 2021 15:06:53 +0800 Subject: [PATCH] Ignore Ramda methods in rules (#1557) Co-authored-by: fisker Cheung --- rules/no-array-for-each.js | 1 + rules/no-array-method-this-argument.js | 5 +++++ test/no-array-method-this-argument.mjs | 5 +++++ 3 files changed, 11 insertions(+) diff --git a/rules/no-array-for-each.js b/rules/no-array-for-each.js index a479fe4bda..1a8cd6026b 100644 --- a/rules/no-array-for-each.js +++ b/rules/no-array-for-each.js @@ -361,6 +361,7 @@ function isFixable(callExpression, {scope, functionInfo, allIdentifiers, context const ignoredObjects = [ 'React.Children', 'Children', + 'R', ]; const create = context => { diff --git a/rules/no-array-method-this-argument.js b/rules/no-array-method-this-argument.js index 537d36f4a5..1302470734 100644 --- a/rules/no-array-method-this-argument.js +++ b/rules/no-array-method-this-argument.js @@ -24,14 +24,17 @@ const ignored = [ '_.filter', 'underscore.filter', 'Vue.filter', + 'R.filter', 'lodash.find', '_.find', 'underscore.find', + 'R.find', 'lodash.findIndex', '_.findIndex', 'underscore.findIndex', + 'R.findIndex', 'lodash.flatMap', '_.flatMap', @@ -40,6 +43,7 @@ const ignored = [ '_.forEach', 'React.Children.forEach', 'Children.forEach', + 'R.forEach', 'lodash.map', '_.map', @@ -48,6 +52,7 @@ const ignored = [ 'Children.map', 'jQuery.map', '$.map', + 'R.map', 'lodash.some', '_.some', diff --git a/test/no-array-method-this-argument.mjs b/test/no-array-method-this-argument.mjs index a46da2a126..3bd10d5048 100644 --- a/test/no-array-method-this-argument.mjs +++ b/test/no-array-method-this-argument.mjs @@ -25,6 +25,11 @@ test.snapshot({ 'React.Children.forEach(children, () => {})', 'Children.forEach(children, () => {})', 'Vue.filter("capitalize", () => {})', + 'R.filter(() => {}, [])', + 'R.find(() => {}, [])', + 'R.findIndex(() => {}, [])', + 'R.forEach(() => {}, [])', + 'R.map(() => {}, [])', // `jQuery.find` and `jQuery.filter` don't accept second argument '$( "li" ).filter( ":nth-child(2n)" ).css( "background-color", "red" );', '$( "li.item-ii" ).find( "li" ).css( "background-color", "red" );',