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
Bump to depend on Airbnb 17.0.1 #27
Conversation
Ignoring max-len for strings and template literals makes it too easy to include long lines in the project. However, if one is using prettier settings with eslint, max-len gets ingored anyway.
@@ -23,7 +23,7 @@ rules: | |||
|
|||
# http://eslint.org/docs/rules/max-len | |||
# https://github.com/airbnb/javascript/blob/master/packages/eslint-config-airbnb-base/rules/style.js#L128 | |||
# Almost identical to airbnb's version, but adds ignore of `import` statements | |||
# Almost identical to airbnb's version, but adds ignore of `import` statements and sets ignoreStrings: false |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think airbnb version is correct?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is Airbnb: https://github.com/airbnb/javascript/blob/17e0454672f784772c5f5ab052523a93229d9adf/packages/eslint-config-airbnb-base/rules/style.js#L195
If you prefer, I'll remove these for now.
// specify the maximum length of a line in your program
// https://eslint.org/docs/rules/max-len
'max-len': ['error', 100, 2, {
ignoreUrls: true,
ignoreComments: false,
ignoreRegExpLiterals: true,
ignoreStrings: true,
ignoreTemplateLiterals: true,
}],
"eslint": "^4.19.1", | ||
"eslint-plugin-import": "^2.12.0", | ||
"eslint-plugin-jsx-a11y": "^6.0.3", | ||
"eslint-plugin-react": "^7.9.1" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
React deps are optional.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I copied these from Airbnb's one.
@alexfedoseev I'm merging. We can do any review comments in a follow-on PR. |
Ignoring max-len for strings and template literals makes it too easy
to include long lines in the project.
However, if one is using prettier settings with eslint, max-len gets
ingored anyway.
This change is