tools: Add no useless regex char class rule by princejwesley · Pull Request #9591 · nodejs/node

@nodejs-github-bot added the tools

Issues and PRs related to the tools directory.

label

Nov 13, 2016

not-an-aardvark

Eslint Rule:
Disallow useless escape in regex character class
with optional override characters option and auto
fixable with eslint --fix option.

Usage:
no-useless-regex-char-class-escape: [2, { override: ['[', ']'] }]

PR-URL: nodejs#9591
The `/` character does not need to be escaped when occurring inside a
character class in a regular expression. Remove such instances of
escaping in the code base.

PR-URL: nodejs#9591

not-an-aardvark

Trott pushed a commit to Trott/io.js that referenced this pull request

Nov 24, 2016
The `/` character does not need to be escaped when occurring inside a
character class in a regular expression. Remove such instances of
escaping in the code base.

PR-URL: nodejs#9591
Reviewed-By: Teddy Katz <teddy.katz@gmail.com>

Trott pushed a commit to Trott/io.js that referenced this pull request

Nov 24, 2016
Eslint Rule:
Disallow useless escape in regex character class
with optional override characters option and auto
fixable with eslint --fix option.

Usage:
no-useless-regex-char-class-escape: [2, { override: ['[', ']'] }]

PR-URL: nodejs#9591
Reviewed-By: Teddy Katz <teddy.katz@gmail.com>

addaleax pushed a commit that referenced this pull request

Dec 5, 2016
The `/` character does not need to be escaped when occurring inside a
character class in a regular expression. Remove such instances of
escaping in the code base.

PR-URL: #9591
Reviewed-By: Teddy Katz <teddy.katz@gmail.com>

addaleax pushed a commit that referenced this pull request

Dec 5, 2016
Eslint Rule:
Disallow useless escape in regex character class
with optional override characters option and auto
fixable with eslint --fix option.

Usage:
no-useless-regex-char-class-escape: [2, { override: ['[', ']'] }]

PR-URL: #9591
Reviewed-By: Teddy Katz <teddy.katz@gmail.com>

MylesBorins pushed a commit that referenced this pull request

May 8, 2017
Eslint Rule:
Disallow useless escape in regex character class
with optional override characters option and auto
fixable with eslint --fix option.

Usage:
no-useless-regex-char-class-escape: [2, { override: ['[', ']'] }]

PR-URL: #9591
Reviewed-By: Teddy Katz <teddy.katz@gmail.com>

andrew749 pushed a commit to michielbaird/node that referenced this pull request

Jul 19, 2017
Eslint Rule:
Disallow useless escape in regex character class
with optional override characters option and auto
fixable with eslint --fix option.

Usage:
no-useless-regex-char-class-escape: [2, { override: ['[', ']'] }]

PR-URL: nodejs/node#9591
Reviewed-By: Teddy Katz <teddy.katz@gmail.com>