Added max-len rule #510
Added max-len rule #510
Conversation
|
I think according to https://github.com/airbnb/javascript#6.2 our max-length is 100 characters, not 80 characters. Please make the length explicit. I thought that this number was already well-declared and obvious in our styleguide, but I think I was thinking of our Ruby styleguide (length guidance here: https://github.com/airbnb/ruby#line-length). Sorry to change direction so fast; I think we should wait for more discussion on the topic before merging this. |
|
Ok. I am fine with 100 characters as well. Should I modify this pull request or should we wait for others to discuss this whole thing? |
README.md
Outdated
There was a problem hiding this comment.
While I don't see a rule in the styleguide about it, in our examples we generally don't quote object keys when they're not invalid identifiers. Can you unquote all of these object keys?
There was a problem hiding this comment.
you should submit a PR adding this rule :)
There was a problem hiding this comment.
Done. And yes: if so then there should be a rule for that 😜
|
Go ahead and make it explicitly 100 chars in your markdown update and in the eslintrc. We'll let the PR sit here for a day or two after that and then I'll merge after consensus/people forget |
|
Done 😎 |
There was a problem hiding this comment.
"qux" is unnecessarily quoted
There was a problem hiding this comment.
this line should be obviously too long, as it is it's 96 characters, make it like 120+ chars
There was a problem hiding this comment.
Oh, I am sorry. This happens when you are in hurry 😕. I will fix that
|
Should I take the description really in the whitespace section? I think it has nothing to do with whitespaces. |
There was a problem hiding this comment.
Also, max-len is already specified in legacy.js
|
+1 I would like to see this merged. |
|
oops |
|
@screendriver if you'd like to rebase this on top of master, we can now add and renumber sections without breaking existing links. |
7 participants
Closes #509