{"action":"created","issue":{"url":"https://api.github.com/repos/spalger/gulp-jshint/issues/106","labels_url":"https://api.github.com/repos/spalger/gulp-jshint/issues/106/labels{/name}","comments_url":"https://api.github.com/repos/spalger/gulp-jshint/issues/106/comments","events_url":"https://api.github.com/repos/spalger/gulp-jshint/issues/106/events","html_url":"https://github.com/spalger/gulp-jshint/pull/106","id":65600580,"number":106,"title":"Ability to specify alternative jshint implementation","user":{"login":"yoni-tock","id":11616747,"avatar_url":"https://avatars.githubusercontent.com/u/11616747?v=3","gravatar_id":"","url":"https://api.github.com/users/yoni-tock","html_url":"https://github.com/yoni-tock","followers_url":"https://api.github.com/users/yoni-tock/followers","following_url":"https://api.github.com/users/yoni-tock/following{/other_user}","gists_url":"https://api.github.com/users/yoni-tock/gists{/gist_id}","starred_url":"https://api.github.com/users/yoni-tock/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/yoni-tock/subscriptions","organizations_url":"https://api.github.com/users/yoni-tock/orgs","repos_url":"https://api.github.com/users/yoni-tock/repos","events_url":"https://api.github.com/users/yoni-tock/events{/privacy}","received_events_url":"https://api.github.com/users/yoni-tock/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":2,"created_at":"2015-04-01T04:08:41Z","updated_at":"2015-04-01T17:02:53Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/spalger/gulp-jshint/pulls/106","html_url":"https://github.com/spalger/gulp-jshint/pull/106","diff_url":"https://github.com/spalger/gulp-jshint/pull/106.diff","patch_url":"https://github.com/spalger/gulp-jshint/pull/106.patch"},"body":"Closes #89. Closes #90.\r\n\r\nI patched this because I wanted a nice way to run jsxhint via gulp. [gulp-jsxhint](https://www.npmjs.com/package/gulp-jsxhint) is a stub implementation that's not yet supported and there were a number of issues in here (#55, #89, #90) that suggest this is a desired feature. @jtangelder suggested a particularly clean solution, which is what I implemented here.\r\n\r\nThe motivation for wanting to actually use jsxhint rather than, as you suggest in #55:\r\n\r\n```js\r\n.pipe react()\r\n.pipe jshint()\r\n```\r\n\r\nis twofold.\r\n\r\nFirst, the conversion from jsx -> js can mess up the line numbers that jshint spits out when finding warnings.\r\n\r\nSecond, the conversion from jsx -> js can cause lint errors to exist in js code that didn't exist in jsx code (this is mentioned by @elierotenberg in #55)\r\n\r\nI'm more than happy to iterate on this as needed."},"comment":{"url":"https://api.github.com/repos/spalger/gulp-jshint/issues/comments/88555680","html_url":"https://github.com/spalger/gulp-jshint/pull/106#issuecomment-88555680","issue_url":"https://api.github.com/repos/spalger/gulp-jshint/issues/106","id":88555680,"user":{"login":"yoni-tock","id":11616747,"avatar_url":"https://avatars.githubusercontent.com/u/11616747?v=3","gravatar_id":"","url":"https://api.github.com/users/yoni-tock","html_url":"https://github.com/yoni-tock","followers_url":"https://api.github.com/users/yoni-tock/followers","following_url":"https://api.github.com/users/yoni-tock/following{/other_user}","gists_url":"https://api.github.com/users/yoni-tock/gists{/gist_id}","starred_url":"https://api.github.com/users/yoni-tock/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/yoni-tock/subscriptions","organizations_url":"https://api.github.com/users/yoni-tock/orgs","repos_url":"https://api.github.com/users/yoni-tock/repos","events_url":"https://api.github.com/users/yoni-tock/events{/privacy}","received_events_url":"https://api.github.com/users/yoni-tock/received_events","type":"User","site_admin":false},"created_at":"2015-04-01T17:02:53Z","updated_at":"2015-04-01T17:02:53Z","body":"@spalger Mostly so that when testing locally, `npm install` would see a version diff and actually install the new code. Happy to remove that commit if that's the blocker! Since it's your project it's obviously 100% your call when to bump the version."}}