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

Add a reportDisables secondary option #4897

Merged
merged 4 commits into from Aug 19, 2020
Merged

Add a reportDisables secondary option #4897

merged 4 commits into from Aug 19, 2020

Commits on Aug 11, 2020

  1. Rename UnusedRange to DisableReportRange and .unusedRule to .rule 

    This allows us to re-use this structure for representing errors
relating to disabled that aren't specifically related to the disable
being unused.
    @nex3
    nex3 committed Aug 11, 2020
    Copy the full SHA
    a97f4f0 View commit details
    Browse the repository at this point in the history

Commits on Aug 13, 2020

  1. Add a reportDisables secondary option 

    This allows config authors to specify that certain rules may not be
disabled by authors.

Closes #4875
    @nex3
    nex3 committed Aug 13, 2020
    Copy the full SHA
    3db150c View commit details
    Browse the repository at this point in the history

  2. Produce different messages for different disable-level failures

    @nex3
    nex3 committed Aug 13, 2020
    Copy the full SHA
    0c2304b View commit details
    Browse the repository at this point in the history

Commits on Aug 17, 2020

  1. Code review

    @nex3
    nex3 committed Aug 17, 2020
    Copy the full SHA
    10b89f4 View commit details
    Browse the repository at this point in the history