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

Fix ESLint indentation warnings #219

Closed
wants to merge 1 commit into from
Closed

Fix ESLint indentation warnings #219

wants to merge 1 commit into from

Conversation

TheDancingCode
Copy link

This properly indents the code to fix 300+ ESLint warnings.

Six max-len warnings remain, but they're less straightforward to resolve, so I leave that up to you.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 99.31% when pulling a21fbee on TheDancingCode:fix-eslint-warnings into a58a58d on adametry:master.

1 similar comment
@coveralls
Copy link

Coverage Status

Coverage remained the same at 99.31% when pulling a21fbee on TheDancingCode:fix-eslint-warnings into a58a58d on adametry:master.

@TheDancingCode
Copy link
Author

@adametry
Did you get a chance to look at this?

@Lamisi
Copy link

Lamisi commented Jun 1, 2018

Is this the same error as this:

142:11 error Expected indentation of 12 space characters but found 10 react/jsx-indent
'lint-app' errored after 5.46 s
ESLintError in plugin 'gulp-eslint'

@TheDancingCode
Copy link
Author

Not exactly. This PR fixes the same kind of error, but then for the code found in this repo.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants