freaktechnik · freaktechnik · May 5, 2025 · May 5, 2025
diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -17,7 +17,7 @@
     "@typescript-eslint/parser": "^8.31.1",
     "ava": "^6.3.0",
     "c8": "^10.1.3",
-    "eslint": "^9.25.1",
+    "eslint": "^9.26.0",
     "eslint-ava-rule-tester": "^5.0.1",
     "eslint-plugin-eslint-plugin": "^6.4.0",
     "eslint-plugin-optimize-regex": "^1.2.1",

diff --git a/test/rules/no-unnecessary-this-arg.mjs b/test/rules/no-unnecessary-this-arg.mjs
 import test from 'ava';
 import AvaRuleTester from 'eslint-ava-rule-tester';
 import rule from '../../rules/no-unnecessary-this-arg.js';

                column: 24,
                line: 1,
            } ],
            output: null,
        },
        {
            code: 'array.some((t) => t.id, b)',
                column: 25,
                line: 1,
            } ],
            output: null,
        },
        {
            code: 'array.filter((t) => t.id, 2)',
                column: 27,
                line: 1,
            } ],
            output: null,
        },
        {
            code: 'array.filter((t) => t.id, null)',
                column: 27,
                line: 1,
            } ],
            output: null,
        },
    ],
 });
diff --git a/rules/no-unnecessary-this-arg.js b/rules/no-unnecessary-this-arg.js
 * @license MIT
 * @author Martin Giger
 */
 import {
    isMethod,
    isOnObject,
 } from "../lib/helpers/call-expression.js";
diff --git a/rules/from-map.js b/rules/from-map.js
                                restParameterString = PARAM_SEPARATOR + restParameters_.join(PARAM_SEPARATOR);
                            }
                            // The original map callback from Array.from gets nested as a parameter to the callback from map.
                            const lastCallback = getCallback(mapCallback, mapThisArgument, `${firstCallback}${restParameterString}`),
                                [
                                    callbackStartLocation
                                    , callbackEndLocation,
                        }

                        // Move the map arguments to from.
                        const [ firstArgument ] = node.arguments,
                            [ argumentStartLocation ] = firstArgument.range,
                            [
                                , parentEndLocation,
diff --git a/rules/avoid-reverse.js b/rules/avoid-reverse.js
                if(Object.keys(REPLACEMENTS).every((m) => !isMethod(node, m))) {
                    return;
                }
                const reversed = REPLACEMENTS[node.callee.property.name];

                context.report({
                    node: node.callee.property,
	import test from 'ava';
Check warning on line 1 in test/rules/no-unnecessary-this-arg.mjs GitHub Actions / test The filename `no-unnecessary-this-arg.mjs` should be named `no-unnecessary-this-argument.mjs`. A more descriptive name will do too Check warning on line 1 in test/rules/no-unnecessary-this-arg.mjs GitHub Actions / test The filename `no-unnecessary-this-arg.mjs` should be named `no-unnecessary-this-argument.mjs`. A more descriptive name will do too
	import AvaRuleTester from 'eslint-ava-rule-tester';
	import rule from '../../rules/no-unnecessary-this-arg.js';

	column: 24,
	line: 1,
	} ],
	output: null,
Check warning on line 37 in test/rules/no-unnecessary-this-arg.mjs GitHub Actions / test Use `undefined` instead of `null` Check warning on line 37 in test/rules/no-unnecessary-this-arg.mjs GitHub Actions / test Use `undefined` instead of `null`
	},
	{
	code: 'array.some((t) => t.id, b)',
	column: 25,
	line: 1,
	} ],
	output: null,
Check warning on line 46 in test/rules/no-unnecessary-this-arg.mjs GitHub Actions / test Use `undefined` instead of `null` Check warning on line 46 in test/rules/no-unnecessary-this-arg.mjs GitHub Actions / test Use `undefined` instead of `null`
	},
	{
	code: 'array.filter((t) => t.id, 2)',
	column: 27,
	line: 1,
	} ],
	output: null,
Check warning on line 55 in test/rules/no-unnecessary-this-arg.mjs GitHub Actions / test Use `undefined` instead of `null` Check warning on line 55 in test/rules/no-unnecessary-this-arg.mjs GitHub Actions / test Use `undefined` instead of `null`
	},
	{
	code: 'array.filter((t) => t.id, null)',
	column: 27,
	line: 1,
	} ],
	output: null,
Check warning on line 64 in test/rules/no-unnecessary-this-arg.mjs GitHub Actions / test Use `undefined` instead of `null` Check warning on line 64 in test/rules/no-unnecessary-this-arg.mjs GitHub Actions / test Use `undefined` instead of `null`
	},
	],
	});
	* @license MIT
	* @author Martin Giger
	*/
	import {
Check warning on line 5 in rules/no-unnecessary-this-arg.js GitHub Actions / test The filename `no-unnecessary-this-arg.js` should be named `no-unnecessary-this-argument.js`. A more descriptive name will do too Check warning on line 5 in rules/no-unnecessary-this-arg.js GitHub Actions / test The filename `no-unnecessary-this-arg.js` should be named `no-unnecessary-this-argument.js`. A more descriptive name will do too
	isMethod,
	isOnObject,
	} from "../lib/helpers/call-expression.js";
	restParameterString = PARAM_SEPARATOR + restParameters_.join(PARAM_SEPARATOR);
	}
	// The original map callback from Array.from gets nested as a parameter to the callback from map.
	const lastCallback = getCallback(mapCallback, mapThisArgument, `${firstCallback}${restParameterString}`),
Check warning on line 98 in rules/from-map.js GitHub Actions / test `Combine this with the previous 'const' statement` Check warning on line 98 in rules/from-map.js GitHub Actions / test `Combine this with the previous 'const' statement`
	[
	callbackStartLocation
	, callbackEndLocation,
	}

	// Move the map arguments to from.
	const [ firstArgument ] = node.arguments,
Check warning on line 117 in rules/from-map.js GitHub Actions / test `Combine this with the previous 'const' statement` Check warning on line 117 in rules/from-map.js GitHub Actions / test `Combine this with the previous 'const' statement`
	[ argumentStartLocation ] = firstArgument.range,
	[
	, parentEndLocation,
	if(Object.keys(REPLACEMENTS).every((m) => !isMethod(node, m))) {
	return;
	}
	const reversed = REPLACEMENTS[node.callee.property.name];
Check warning on line 33 in rules/avoid-reverse.js GitHub Actions / test `Combine this with the previous 'const' statement` Check warning on line 33 in rules/avoid-reverse.js GitHub Actions / test `Combine this with the previous 'const' statement`

	context.report({
	node: node.callee.property,