search

jscs-dev / node-jscs

:arrow_heading_up: JavaScript Code Style checker (unmaintained)
https://jscs-dev.github.io
MIT License
4.97k stars 515 forks source link

disallowObjectKeysOnNewLine seems not to do what it says #2229

Closed bcowgill closed 8 years ago

bcowgill commented 8 years ago

with disallowObjectKeysOnNewLine: true

disallowObjectKeysOnNewLine: Object keys must go on a new line at src/templates/jscs-test.jsx :
   514 |//Invalid
   515 |var temp1 = {
   516 |    temp2: 'b',
-----------------------^
   517 |    temp3: 'c'
   518 |};

Shouldn't it be disallowObjectKeysOnSameLine ?

hzoo commented 8 years ago

There is also http://jscs.info/rule/requireObjectKeysOnNewLine, so it looks like the message just needs to be changed here