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

Chore: lint all files in the repo at the same time #11425

Merged
merged 2 commits into from Feb 21, 2019
Merged

Chore: lint all files in the repo at the same time #11425

merged 2 commits into from Feb 21, 2019

Commits on Feb 21, 2019

  1. Chore: lint all files in the repo at the same time 

    Previously, `npm run lint` would invoke ESLint three separate times for different sets of files. There doesn't seem to have been a clear reason for this (it was added in 9b8c639 about 5 years ago), and some files in the repo were inadvertently being missed. This commit updates the lint task to simply invoke ESLint once on the entire repository. It also fixes some linting errors in files that were previously being skipped.
    @not-an-aardvark
    not-an-aardvark committed Feb 21, 2019
    Copy the full SHA
    6878ca5 View commit details
    Browse the repository at this point in the history

  2. Remove js-yaml from eslint-config-eslint dependencies

    @not-an-aardvark
    not-an-aardvark committed Feb 21, 2019
    Copy the full SHA
    c1d4210 View commit details
    Browse the repository at this point in the history