Revert isNodeExecutable: yes

[arturparkhisenko]

No description provided.

[arturparkhisenko]

ex path its when i see last csslint commit :D

[druellan]

Just to point out that on Atom 0.183 and CSSLINT 0.0.11, I need this patch to get it working on Windows 8.1

[rustamli]

Guys please merge this one, otherwise it doesn't work on Windows. Had to spend sometime before I figured out that it wasn't working because of isNodeExecutable

[arturparkhisenko]

@AsaAyers @dhagerty9009 @relekang @k2b6s9j someone pls..

[keplersj]

I have no pulling power here. You're looking for @hd-demam probably.

[keplersj]

/cc @hd-deman

Damn GitHub mobile.

[arturparkhisenko]

@k2b6s9j thank you 👍

[keplersj]

@ikeagold You're quite welcome.

[Arcanemagus]

@hd-deman, @kevinsawicki, @zimme, @pborreli:
Can anyone merge this PR so this plugin works properly without modification?

[ansballard]

👍

[corintho]

After making this change on my local installation, I was able to have it properly running in windows. So, a +1 to the fix. And a thank you for ikeagold for spotting the issue and opening the pull request for all of us.

[keplersj]

So the real solution here isn't to revert isNodeExecutable, it's for this linter to be rewritten for the new Linter 1.0 API. I'll merge this in for now since so many of you are claiming success but this linter is on my queue for linters which absolutely need a rewrite.

[keplersj]

This change has been published in v0.0.14

[arturparkhisenko]

@k2b6s9j :) this is must be a best code n text editor in a world!