Move reporters to lib/reporters and expose nodelint --{name}-reporter option for each
Sannis opened this issue · comments
Start of discussion: #20
@mkitt, are we need keep both elaborate.js and syntastic-reporter.js for vim?
I don't think so. I believe they are in there as just examples for users who might be interested in integrating into an editor or creating a reporter. Feel free to blow them away.
Thanks for quick response. I'll try to use default and these reporters with vim and remove duplicates.
I'l keep both TextMate reporters with their names.
I've rebase by fork to make commits history more clean. All tests passes (http://travis-ci.org/#!/Sannis/nodelint/builds), reporters moved to lib/reporters and available by reporter name or by filename.
@cliffano: Now built-in reporters available with --reporter=name (f.e. --reporter=xml), so does it necessary to have --xml-reporter cli option?
Sorry for late reply. Nope, happy with the --reporter=name approach :). Thanks for adding it.
No problems :) I was a little sick, so it will not finished until next week.
Merged.