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

fix lint warnings caused by recent ESLint configuration update #5031

Closed
wants to merge 2 commits into from
Closed

fix lint warnings caused by recent ESLint configuration update #5031

wants to merge 2 commits into from

Commits on Jun 17, 2020

  1. lint for the lint gods 

    * fix trailing comma and indentation warnings
* turn off typescript-eslint indent rule (to workaround myriad issues)
* add manual single line ignore (i _think_ that is the right move 🤷🏼‍♂️ anyway)
    @jgravois
    jgravois committed Jun 17, 2020
    Configuration menu
    Copy the full SHA
    354ab15 View commit details
    Browse the repository at this point in the history

  2. instead of a custom rule, make the change upstream 

    relies on sveltejs/eslint-config#1
    @jgravois
    jgravois committed Jun 17, 2020
    Configuration menu
    Copy the full SHA
    52eee5e View commit details
    Browse the repository at this point in the history