`no-restricted-imports`: disable comments do not work for individual named imports

Tell us about your environment

• ESLint Version: 5.16.0
• Node Version: 10.16.3
• npm Version: 6.11.3

What parser (default, Babel-ESLint, etc.) are you using?

Please show your full configuration:

Configuration

{
  rules: {
    'no-restricted-imports': [
      'error',
      {
        name: 'lodash',
        importNames: ['zip'],
      },
    ],
  },
};

What did you do? Please include the actual source code causing the issue, as well as the command that you used to run ESLint.

import {
  zip, // eslint-disable-line no-restricted-imports
  pickBy,
} from 'lodash';

eslint

What did you expect to happen? No error

What actually happened? Please include the actual, raw output from ESLint. Error

'lodash' import is restricted from being used.

To workaround this you must disable the rule for the whole import statement. However this is not ideal because this could accidentally silence errors that we want to hear about, for other named imports:

// eslint-disable-next-line no-restricted-imports
import {
  zip,
  pickBy,
} from 'lodash';

Are you willing to submit a pull request to fix this bug?

[fanich37]

I’m gonna check it and send a PR till New Year.

[mdjermanovic]

It seems that reporting zip identifier node instead of the whole import statement would fix this issue. Would be probably nice to have a different error message in this case.

This change also means that the rule can now report multiple errors for the same import statement.