-
Notifications
You must be signed in to change notification settings - Fork 26.3k
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
Misleading documentation 10.8 #2206
Comments
The eslint link should not give you any confidence that the linter will cover the entire guideline - you always have to keep the entire guide in your head, just in case. javascript/packages/eslint-config-airbnb-base/rules/style.js Lines 396 to 401 in 06b3ab1
|
Unless I'm missing something I think this may be a bug. I just tested this out again with the following:
6:3 warning Unexpected console statement no-console ✖ 1 problem (0 errors, 1 warning) |
In that case, please file it on eslint itself. |
I think the "consistent" rule is satisfied if all of the properties are on the same line, even if the braces aren't. If you have a newline in the middle of the list, what happens? |
The documentation for 10.8 references
object-curly-newline
however object-curly-newline does not catch this rule. Furthermore airbnb's set of rules also doesn't catch this rule.See Example
The guidance is good, however the documentation leads you to believe the linter will catch you if you put imports on the same line, which it doesn't.
The text was updated successfully, but these errors were encountered: