search

brackets-userland / brackets-eslint

Adds ESLint support to Brackets
MIT License
43 stars 15 forks source link

Better/OK to install this via Brackets Extension Mgr or npm? #64

Open tonybrasunas-sighten opened 7 years ago

tonybrasunas-sighten commented 7 years ago

Hi, and thanks for your great work on these extensions.

I notice that this eslint extension is now available on the extensions manager. Is there any reason not to use that and to use the npm installer instead? If so, perhaps an update to the readme? I can make a PR if that's something you'd appreciate.

By the way, I had a little trouble installing the npm installer (everything went well apparently, but no icon in the upper right in brackets).

tonybrasunas-sighten commented 7 years ago

Never mind, I missed one of the commands -- to cd into the new directory before running npm install 🤦‍♂️

(Point/Question about using the built-in Extension Manager remains)

zaggino commented 7 years ago

Hi, npm should always have the latest version.

The extensions manager didn't support npm dependencies (eslint) before, now it doesn't always work. I try to upload new versions there if I can but going the npm way or cloning here from github is a safer option to have the latest code.