Suggested fix for `no-div-regex` trips `no-useless-escape`

Tell us about your environment

• ESLint Version: 5.13
• Node Version: 10.15.0
• npm Version: 6.4.1

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

Please show your full configuration:

Configuration

{
	"rules": {
		"no-div-regex": "error",
		"no-useless-escape": "error"
	}
}

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

console.log('test=='.replace(/=*$/, '')); // no-div-regex error
console.log('test=='.replace(/\=*$/, '')); // no-useless-escape error
console.log('test=='.replace(/[=]*$/, '')); // accepted by both rules

eslint script.js

What did you expect to happen? no-div-regex should suggest replacing /=foo/ with /[=]foo/ to avoid no-useless-escape error. Alternatively no-useless-escape could accept an ignoreDivRegex boolean option to allow /\=foo/.

Are you willing to submit a pull request to fix this bug? I’m sure I can handle the documentation approach of suggesting /[=]foo/. I’m not confident I can create the no-useless-escape option to ignore div regex.

[SalsabeelQandeel]

I can work on this one, Is it available?

[coreyfarrell]

@platinumazure thanks for responding. Unfortunately I don’t currently have time to work on this and since xo has removed the no-div-regex this is no longer as issue for me.