jscs-dev / node-jscs

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

[wip] Attempt to reproduce conditions for #2161 #2162

Closed indexzero closed 8 years ago

indexzero commented 8 years ago

:warning: :hand: Not ready for merge yet. Attempting to reproduce conditions for #2161 :hand: :warning:

cc/ @samshull @DullReferenceException

DullReferenceException commented 8 years ago

Is there a 1:1 correlation between reported errors and autofix? I'm not so sure the style checker reports problems, but the autofixer will add a space in one run, then remove it in the next.

indexzero commented 8 years ago

@DullReferenceException I believe so, yes. With auto-fix enabled it would suppress the error and attempt to fix it.

samshull commented 8 years ago

Actually, after reviewing, allExcept:['shorthand'] would allow this to pass as expected in es6. You should close this issue.