search

jednano / eclint

Validate or fix code that doesn't adhere to EditorConfig settings or infer settings from existing code.
MIT License
307 stars 28 forks source link

Publish on NPM #19

Closed mcandre closed 9 years ago

mcandre commented 9 years ago

As a Node.js newbie, I want eclint published on NPM, so that it's easy to install with npm [-g] eclint.

jednano commented 9 years ago

Unfortunately, it doesn't even work yet and it never will because it's impossible to interpret a developer's intent when spaces are used for both indentation and alignment. http://jedmao.ghost.io/2014/08/20/tabs-vs-spaces-the-age-old-war/

mcandre commented 9 years ago

eclint could ignore that rule for now (even editorconfig doesn't support every rule in Google Style or Mozilla Style).

jednano commented 9 years ago

I'm still trying to figure out the best strategy for that. Even still though, the rest of the rules aren't even complete yet. It's not an issue so much as it is an incomplete project.

jednano commented 9 years ago

@mcandre you can look forward to a 0.1.0 release, probably later tonight. I just need to update the readme with all the necessary instructions. See the milestones.