{"action":"created","issue":{"url":"https://api.github.com/repos/yannickcr/eslint-plugin-react/issues/611","repository_url":"https://api.github.com/repos/yannickcr/eslint-plugin-react","labels_url":"https://api.github.com/repos/yannickcr/eslint-plugin-react/issues/611/labels{/name}","comments_url":"https://api.github.com/repos/yannickcr/eslint-plugin-react/issues/611/comments","events_url":"https://api.github.com/repos/yannickcr/eslint-plugin-react/issues/611/events","html_url":"https://github.com/yannickcr/eslint-plugin-react/pull/611","id":156188473,"number":611,"title":"Adds new `unused-prop-types` rule","user":{"login":"EvNaverniouk","id":226640,"avatar_url":"https://avatars.githubusercontent.com/u/226640?v=3","gravatar_id":"","url":"https://api.github.com/users/EvNaverniouk","html_url":"https://github.com/EvNaverniouk","followers_url":"https://api.github.com/users/EvNaverniouk/followers","following_url":"https://api.github.com/users/EvNaverniouk/following{/other_user}","gists_url":"https://api.github.com/users/EvNaverniouk/gists{/gist_id}","starred_url":"https://api.github.com/users/EvNaverniouk/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/EvNaverniouk/subscriptions","organizations_url":"https://api.github.com/users/EvNaverniouk/orgs","repos_url":"https://api.github.com/users/EvNaverniouk/repos","events_url":"https://api.github.com/users/EvNaverniouk/events{/privacy}","received_events_url":"https://api.github.com/users/EvNaverniouk/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/yannickcr/eslint-plugin-react/labels/enhancement","name":"enhancement","color":"84b6eb"},{"url":"https://api.github.com/repos/yannickcr/eslint-plugin-react/labels/new%20rule","name":"new rule","color":"eb6420"}],"state":"open","locked":false,"assignee":{"login":"yannickcr","id":13209,"avatar_url":"https://avatars.githubusercontent.com/u/13209?v=3","gravatar_id":"","url":"https://api.github.com/users/yannickcr","html_url":"https://github.com/yannickcr","followers_url":"https://api.github.com/users/yannickcr/followers","following_url":"https://api.github.com/users/yannickcr/following{/other_user}","gists_url":"https://api.github.com/users/yannickcr/gists{/gist_id}","starred_url":"https://api.github.com/users/yannickcr/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/yannickcr/subscriptions","organizations_url":"https://api.github.com/users/yannickcr/orgs","repos_url":"https://api.github.com/users/yannickcr/repos","events_url":"https://api.github.com/users/yannickcr/events{/privacy}","received_events_url":"https://api.github.com/users/yannickcr/received_events","type":"User","site_admin":false},"assignees":[{"login":"yannickcr","id":13209,"avatar_url":"https://avatars.githubusercontent.com/u/13209?v=3","gravatar_id":"","url":"https://api.github.com/users/yannickcr","html_url":"https://github.com/yannickcr","followers_url":"https://api.github.com/users/yannickcr/followers","following_url":"https://api.github.com/users/yannickcr/following{/other_user}","gists_url":"https://api.github.com/users/yannickcr/gists{/gist_id}","starred_url":"https://api.github.com/users/yannickcr/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/yannickcr/subscriptions","organizations_url":"https://api.github.com/users/yannickcr/orgs","repos_url":"https://api.github.com/users/yannickcr/repos","events_url":"https://api.github.com/users/yannickcr/events{/privacy}","received_events_url":"https://api.github.com/users/yannickcr/received_events","type":"User","site_admin":false}],"milestone":null,"comments":19,"created_at":"2016-05-23T02:17:40Z","updated_at":"2016-08-08T16:55:40Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/yannickcr/eslint-plugin-react/pulls/611","html_url":"https://github.com/yannickcr/eslint-plugin-react/pull/611","diff_url":"https://github.com/yannickcr/eslint-plugin-react/pull/611.diff","patch_url":"https://github.com/yannickcr/eslint-plugin-react/pull/611.patch"},"body":"Fixes #226 by adding an new `unused-prop-types` rule.\r\n\r\nThe rule is based very heavily on the `prop-types` rule and shares a lot of the same code. For the time being, I have not made any attempts at breaking out the shared code into a reusable utility file. Please advise on the best way to do this. I was simply thinking of dropping a `proptypes.js` file into the `utils` directory with all the shared functions. If that's acceptable, let me know and I will do that.\r\n\r\nHave tested against several React projects with great success so far. No false positive problems yet, but it needs a lot more testing against other repos. I'm hoping other users will report issues as they try it. For my needs, this rule is working great.\r\n\r\nCan be tested by following these steps:\r\n\r\n```\r\nnpm install EvNaverniouk/eslint-plugin-react#6d309e13a65a1032577ae0c98081d778f85d515a\r\n```\r\n\r\nThen enable the rule in your eslintrc:\r\n\r\n```\r\n\"rules\": {\r\n    \"react/unused-prop-types\": 2\r\n}\r\n```"},"comment":{"url":"https://api.github.com/repos/yannickcr/eslint-plugin-react/issues/comments/238299556","html_url":"https://github.com/yannickcr/eslint-plugin-react/pull/611#issuecomment-238299556","issue_url":"https://api.github.com/repos/yannickcr/eslint-plugin-react/issues/611","id":238299556,"user":{"login":"dhruska","id":2578537,"avatar_url":"https://avatars.githubusercontent.com/u/2578537?v=3","gravatar_id":"","url":"https://api.github.com/users/dhruska","html_url":"https://github.com/dhruska","followers_url":"https://api.github.com/users/dhruska/followers","following_url":"https://api.github.com/users/dhruska/following{/other_user}","gists_url":"https://api.github.com/users/dhruska/gists{/gist_id}","starred_url":"https://api.github.com/users/dhruska/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/dhruska/subscriptions","organizations_url":"https://api.github.com/users/dhruska/orgs","repos_url":"https://api.github.com/users/dhruska/repos","events_url":"https://api.github.com/users/dhruska/events{/privacy}","received_events_url":"https://api.github.com/users/dhruska/received_events","type":"User","site_admin":false},"created_at":"2016-08-08T16:55:40Z","updated_at":"2016-08-08T16:55:40Z","body":"I pulled down this branch into one of my projects to give it a try, since I'm really excited about this - it worked great, except for in one component where it did not detect that I was only using a prop in `componentWillReceiveProps` on the `nextProps` object. Granted, it's probably a poor design pattern in my project that I am doing this, but the linter threw an error when I was indeed using the prop."}}
{"actor":{"display_login":"dhruska"}}