eslint-plugin-sonarjs icon indicating copy to clipboard operation
eslint-plugin-sonarjs copied to clipboard

no-extra-arguments not working with imported functions

Open brightsider opened this issue 2 years ago • 3 comments

I want to report a bug.

Reproducer

// minimal reproducer if relevant
// file A.js
export const createStatus = (params) => UserStatus.create(params);
// file B.js
const userStatusDao = require('./A');
await userStatusDao.createStatus(data, { transaction: t });

Expected behavior I need error for this case, but it's ignored.

eslint-plugin-sonarjs version: 0.23
eslint version: 8.53.0
Node.js version: 16.20.2

Rule key: no-extra-arguments

brightsider avatar Nov 14 '23 06:11 brightsider

Hello @brightsider,

Thanks for reporting this. Indeed, the rule is not currently able to do that.

We'll discuss this improvement internally.

vdiez avatar Nov 15 '23 16:11 vdiez

Any updates on the plans for this feature? I see it's milestoned as 0.23.0, and 0.23.0 is released without that feature.

frederikprijck avatar Jan 26 '24 13:01 frederikprijck

Hi @frederikprijck,

sorry for the late response. I moved this ticket to 0.25, since 0.23 and 0.24 did not contain rules hardening.

vdiez avatar Feb 16 '24 11:02 vdiez

This issue has been migrated to Jira. ESLINTJS-28

Wohops avatar Apr 29 '24 08:04 Wohops