Skip to content
This repository has been archived by the owner on Aug 7, 2023. It is now read-only.

Add support for eslint_d #215

Closed
roman01la opened this issue Sep 20, 2015 · 3 comments
Closed

Add support for eslint_d #215

roman01la opened this issue Sep 20, 2015 · 3 comments

Comments

@roman01la
Copy link

eslint_d support just landed in SublimeLinter-eslint.
Can we have it in this package?
Maybe like another option: Use eslint_d.

@rosston
Copy link
Contributor

rosston commented Sep 20, 2015

There should be little to no performance benefit to using eslint_d over the current method. eslint_d's key performance gain is in not starting up a new node process, but this package already avoids that problem by using eslint's CLIEngine module from within Atom's (already running) node process.

@roman01la
Copy link
Author

@rosston Didn't know about that. Thanks!

@mantoni
Copy link

mantoni commented Sep 21, 2015

Yes, this is important to know. I've added a note for Atom users to the eslint_d readme.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants