73rhodes / Sublime-JSLint

JSLint for Sublime Text 2 and 3.
80 stars 14 forks source link

Move build settings to user settings. Add js_lint_folder command. #11

Closed janraasch closed 11 years ago

janraasch commented 11 years ago

I really like your plugin, but the problem about using a .sublime-build is that though a user can edit the build settings in the Packages/JSLint/JSLint.sublime-build file, but those changes maybe overwritten any time the plugin is updated.

So:

I do realize that this is basically a whole new plugin (codewise at least), but I don't want to add another sublimetext-jslint plugin to the package control repo, since there are already two of them. So, if you'd like to pull this stuff into your repo, please go ahead.

darrenderidder commented 11 years ago

Awesome. Thanks!

may-tau commented 10 years ago

so how do I augment the .sublime-build file without getting it overwritten by each plugin-update? Where am I supposed to do it? Putting a custom JSLint.sublime-build to /Packages/User folder doesn't do the trick.

may-tau commented 10 years ago

UPD: I've found out it's done in .sublime-settings file (by looking into the .sublime-settings file here on github). I don't have that in my local .sublime-settings file though, my standard installed one contains only this:

{
    "filename_filter": "(\\.js|\\.css|\\.html|\\.json|\\.sass|\\.less)$"
}

Seems like the PackageControl doesn't load the new version of the plugin. Any idea why?

may-tau commented 10 years ago

UPD 2: ok, it's actually effective only in @janraasch tree.. why is this still not in the master branch?

janraasch commented 10 years ago

There was a problem with Windows and node-jslint.