Skip to content
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

Test JSHint new features #5759

Closed
1 task done
aeschli opened this issue Apr 25, 2016 · 3 comments
Closed
1 task done

Test JSHint new features #5759

aeschli opened this issue Apr 25, 2016 · 3 comments
Assignees
Milestone

Comments

@aeschli
Copy link
Contributor

aeschli commented Apr 25, 2016

We recently updated the jshint extension with extends and exclude support as described in the extensions ReadMe.md file.

No cross-platform testing required. A single platform suffices.

@aeschli aeschli added this to the April 2016 milestone Apr 25, 2016
@egamma egamma mentioned this issue Apr 25, 2016
68 tasks
@waderyan
Copy link

OK. works.

{
  "jshint.enable": true"
}

Doesn't work.

{
  "jshint.exclude": {
       "./src/data.js": true,
       "Gruntfile.js": true
  }
}

image

Hovering over the green squiggle will show a jshint issue.

@kieferrm
Copy link
Member

The exclude patterns are currently not relative to the workspace root. It makes sense to change that.

@waderyan
Copy link

OK. Resolved. Talked to @kieferrm about a couple of things he has on his roadmap.

@vscodebot vscodebot bot locked and limited conversation to collaborators Nov 18, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

3 participants