polygonplanet / sublime-text-eslint

ESLint any JavaScript file in Sublime Text
24 stars 7 forks source link

Fix to use current working directory's node_modules/eslint in an active project #20 #21

Closed jjdltc closed 6 years ago

jjdltc commented 6 years ago

Fix to use current working directory's node_modules/eslint in an active project #20 Notes:

polygonplanet commented 6 years ago

We confirmed working fine, thank you!

polygonplanet commented 6 years ago

@sebalbisu

This make the plugin depend on Node 8.9.0 or above

I checked this fix in node 8.12.0 and 10.11.0. may be your node' version not match.

jjdltc commented 6 years ago

@polygonplanet How about a compatibility version, that use the global library by default if is under the node version we support. The only problem I see with that, is the lost of the local lib option under that node version (8.9.0).

polygonplanet commented 6 years ago

@jjdltc

a compatibility version, that use the global library by default if is under the node version we support.

I think good.