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.

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

chore: update to latest eslint and linting rules #3995

Merged
merged 1 commit into from Nov 4, 2021

Commits on Nov 4, 2021

  1. chore: update to latest eslint and linting rules

    This brings us in line with the rest of the linting rules we are wanting
    to use on the npm cli repos.
    
    There were several hundred over-length lines and instead of editing them
    all by hand I piped the failing file through `prettier` and back through
    `eslint` just to save some time and energy.  This means there may be
    some quirks in the linting those files have, but we can fix those if we
    see them and they bother us.
    
    Other than that there were about 50 lines that are legitimately
    over-length, all are now explicitly overridden.  Many are tests that
    could be snapshots.
    
    PR-URL: #3995
    Credit: @wraithgar
    Close: #3995
    Reviewed-by: @lukekarrys
    wraithgar committed Nov 4, 2021
    Copy the full SHA
    2256454 View commit details
    Browse the repository at this point in the history