Add an `ignore` option to `no-unused-expressions`

What rule do you want to change? no-unused-expressions

Does this change cause the rule to produce more or fewer warnings? Fewer

How will the change be implemented? (New option, new default behavior, etc.)? New option ignore which takes objects’ names.

Please provide some example code that this change will affect:

import { expect } from 'chai';
const output = 1;
expect(output).to.be.NaN;

What does the rule currently do for this code? Throws a no-unused-expressions error on the last line.

What will the rule do after it’s changed?

/*eslint no-unused-expressions: ["error", { "ignore": ['expect'] }]*/
import { expect } from 'chai';
const output = 1;
expect(output).to.be.NaN;
// => no warning

[ihordiachenko]

@eTorAken I’ve made a small plugin that overrides default no-unused-expressions rule and makes it friendly towards chai expect statements. I hope this will help you.

[not-an-aardvark]

Thanks for your interest in improving ESLint. Unfortunately, it looks like this issue didn’t get enough support from the team and so I’m closing it. While we wish we’d be able to accommodate everyone’s requests, we do need to prioritize. We’ve found that issues failing to reach consensus after a long time tend to never do it, and as such, we close those issues. This doesn’t mean the idea isn’t interesting, just that it’s not something the team can commit to.