Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

tools: Add no useless regex char class rule #9591

Closed
wants to merge 2 commits into from
Closed

tools: Add no useless regex char class rule #9591

wants to merge 2 commits into from

Commits on Nov 14, 2016

  1. tools: Add no useless regex char class rule 

    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
    @princejwesley
    princejwesley committed Nov 14, 2016
    Configuration menu
    Copy the full SHA
    50b8434 View commit details
    Browse the repository at this point in the history

  2. lib,tools: remove unneeded escaping of / 

    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
    @princejwesley
    princejwesley committed Nov 14, 2016
    Configuration menu
    Copy the full SHA
    1fd4f71 View commit details
    Browse the repository at this point in the history