-
Notifications
You must be signed in to change notification settings - Fork 56
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update Standard Dependency #124
Comments
Hey @tyrsius I could update, but as of |
Ah, that's probably why atom starts throwing errors when i |
Related issue: standard/standard#574 |
@ricardofbarros Any reason not to keep the |
Hey @feross, I wanted to move out from having standard, semistandard, etc. as dependencies of this package, because some times the windows users have some problems installing this package because of the path size. I think the issue of atom package's having a long path is already widespread and it's being discussed actively. My idea was to slowly move to the approach of using the projects node_modules standard bin or if there isn't a node_modules directory check the user's PATH for the standard, semistandard, etc. |
Sounds like a reasonable approach. I think it would be better to remove the |
I believe the windows long path issues has been fixed some time ago with newer npm releases. I like the idea to prefer my local node modules. But remove it entirely? I don't think that's a good idea, since we have to install standardjs global to be able use it for non nodejs javascript code, right? |
I've updated all linter and custom parsers deps on the latest release. |
Standard is looking to move to version 8 soon. This package is still on
5.4.1
.Whats the timeline for updating this package to keep pace with standard?
The text was updated successfully, but these errors were encountered: