Fix .eslintrc.json indentation, 2 spaces, not tabs#1029
Fix .eslintrc.json indentation, 2 spaces, not tabs#1029ntwb wants to merge 1 commit intoWordPress:masterfrom ntwb:fix/eslintrc-indentation
.eslintrc.json indentation, 2 spaces, not tabs#1029Conversation
|
I would prefer that we avoid exceptions to coding standards in general. So I don't see a great reason to prefer 2-space indentation over tab indentation for these types of files (*). The 2-spaces rule appears to date back to the introduction of the
(*) Granted this does make sense for I'm also not sure exactly what |
|
I don't remember why it was included in original That said though, it is a current WordPress coding standard to have all
p.s Tabs or spaces are fine based on this: > https://stackoverflow.com/questions/18698738/what-is-the-json-indentation-level-convention |
'til recently this was the case, but npm 5 will now detect indentation:
In the case of the Travis configuration, last I checked it fails spectacularly when using tabs, hence the need for two-spacing. |
|
After thinking on this for a bit, I'm leaning towards option 2, removing |
Follow up to #1022 (comment)