{"action":"created","issue":{"url":"https://api.github.com/repos/facebook/react/issues/3985","labels_url":"https://api.github.com/repos/facebook/react/issues/3985/labels{/name}","comments_url":"https://api.github.com/repos/facebook/react/issues/3985/comments","events_url":"https://api.github.com/repos/facebook/react/issues/3985/events","html_url":"https://github.com/facebook/react/pull/3985","id":82526742,"number":3985,"title":"Enable linting for __tests__","user":{"login":"PiPeep","id":180404,"avatar_url":"https://avatars.githubusercontent.com/u/180404?v=3","gravatar_id":"","url":"https://api.github.com/users/PiPeep","html_url":"https://github.com/PiPeep","followers_url":"https://api.github.com/users/PiPeep/followers","following_url":"https://api.github.com/users/PiPeep/following{/other_user}","gists_url":"https://api.github.com/users/PiPeep/gists{/gist_id}","starred_url":"https://api.github.com/users/PiPeep/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/PiPeep/subscriptions","organizations_url":"https://api.github.com/users/PiPeep/orgs","repos_url":"https://api.github.com/users/PiPeep/repos","events_url":"https://api.github.com/users/PiPeep/events{/privacy}","received_events_url":"https://api.github.com/users/PiPeep/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/facebook/react/labels/GH%20Review:%20review-needed","name":"GH Review: review-needed","color":"fbca04"}],"state":"open","locked":false,"assignee":{"login":"zpao","id":8445,"avatar_url":"https://avatars.githubusercontent.com/u/8445?v=3","gravatar_id":"","url":"https://api.github.com/users/zpao","html_url":"https://github.com/zpao","followers_url":"https://api.github.com/users/zpao/followers","following_url":"https://api.github.com/users/zpao/following{/other_user}","gists_url":"https://api.github.com/users/zpao/gists{/gist_id}","starred_url":"https://api.github.com/users/zpao/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/zpao/subscriptions","organizations_url":"https://api.github.com/users/zpao/orgs","repos_url":"https://api.github.com/users/zpao/repos","events_url":"https://api.github.com/users/zpao/events{/privacy}","received_events_url":"https://api.github.com/users/zpao/received_events","type":"User","site_admin":false},"milestone":null,"comments":1,"created_at":"2015-05-29T18:03:25Z","updated_at":"2015-05-29T18:04:40Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/facebook/react/pulls/3985","html_url":"https://github.com/facebook/react/pull/3985","diff_url":"https://github.com/facebook/react/pull/3985.diff","patch_url":"https://github.com/facebook/react/pull/3985.patch"},"body":"Closes #3971.\r\n\r\n> After #3968, the next thing we should do is start linting our tests.\r\n> Historically we've ignored them due to lack of parser compatibility.\r\n> But that shouldn't be a problem anymore. We may want to integrate\r\n> https://www.npmjs.com/package/eslint-plugin-react to more aggressively\r\n> lint our JSX in tests.\r\n\r\nI understand this diff touches a lot of stuff, so I tried to keep it to\r\na near-minimal set of changes to make eslint happy. If you're wondering\r\nwhy I changed something, let me know and I'll try to map it up to the\r\nrule that I was fixing it for."},"comment":{"url":"https://api.github.com/repos/facebook/react/issues/comments/106891895","html_url":"https://github.com/facebook/react/pull/3985#issuecomment-106891895","issue_url":"https://api.github.com/repos/facebook/react/issues/3985","id":106891895,"user":{"login":"PiPeep","id":180404,"avatar_url":"https://avatars.githubusercontent.com/u/180404?v=3","gravatar_id":"","url":"https://api.github.com/users/PiPeep","html_url":"https://github.com/PiPeep","followers_url":"https://api.github.com/users/PiPeep/followers","following_url":"https://api.github.com/users/PiPeep/following{/other_user}","gists_url":"https://api.github.com/users/PiPeep/gists{/gist_id}","starred_url":"https://api.github.com/users/PiPeep/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/PiPeep/subscriptions","organizations_url":"https://api.github.com/users/PiPeep/orgs","repos_url":"https://api.github.com/users/PiPeep/repos","events_url":"https://api.github.com/users/PiPeep/events{/privacy}","received_events_url":"https://api.github.com/users/PiPeep/received_events","type":"User","site_admin":false},"created_at":"2015-05-29T18:04:40Z","updated_at":"2015-05-29T18:04:40Z","body":"Potential future steps:\r\n\r\n- Enable more of the eslint-plugin-react rules.\r\n- Make our grunt task smarter to be able to pattern-match `__tests__`, and provide alternate .eslintrc files for tests and normal code. E.g. I'd propose disabling `no-unused-expressions` for tests.\r\n- Fix all of the 80-column warnings (ugh)"}}