search

microsoft / vscode-jshint

VSCode extension to integrate jshint into VSCode
https://marketplace.visualstudio.com/items?itemName=dbaeumer.jshint
MIT License
72 stars 48 forks source link

jshint.exclude for internal paths #46

Closed Spown closed 6 years ago

Spown commented 7 years ago

because this issue will probably take literally years to resolve (it's been baried in the backlog for months now) I thought I could find a workaround and ignore the faulty files for this extension. Yet, no matter what glob I try I can't seem do it, probably because of the pattern relativity to the Workroot (#28) and the "file's" path starting with "git:/" which doesn't map to anything. I don't even know what the "real" path is: I tried everything

*.js.git
**.js.git
**/*.js.git
**/**.js.git
**/*.js.git*
**/*.js.git**
**/**.js.git**

tried both with / and \\.

I know it's technically not your bug, but I would be very grateful if you could help. Incorporating an internal filter that only allows actual existent files to be linted wouldn't harm to have anyway. Or maybe at least writable files (these .js.git things are readonly) so that new not yet saved files could be still linted.

Thank you.

RMacfarlane commented 6 years ago

@Spown I just checked in a fix for this. I'll publish a new version of the extension by the end of the week

RMacfarlane commented 6 years ago

Version 0.10.17 is published