{"action":"created","issue":{"url":"https://api.github.com/repos/zulip/zulip/issues/9696","repository_url":"https://api.github.com/repos/zulip/zulip","labels_url":"https://api.github.com/repos/zulip/zulip/issues/9696/labels{/name}","comments_url":"https://api.github.com/repos/zulip/zulip/issues/9696/comments","events_url":"https://api.github.com/repos/zulip/zulip/issues/9696/events","html_url":"https://github.com/zulip/zulip/pull/9696","id":329952292,"node_id":"MDExOlB1bGxSZXF1ZXN0MTkzMDc4NjA0","number":9696,"title":"[Try] eslint: Enable `no-extra-parens` rule.","user":{"login":"shubhamdhama","id":22238472,"node_id":"MDQ6VXNlcjIyMjM4NDcy","avatar_url":"https://avatars0.githubusercontent.com/u/22238472?v=4","gravatar_id":"","url":"https://api.github.com/users/shubhamdhama","html_url":"https://github.com/shubhamdhama","followers_url":"https://api.github.com/users/shubhamdhama/followers","following_url":"https://api.github.com/users/shubhamdhama/following{/other_user}","gists_url":"https://api.github.com/users/shubhamdhama/gists{/gist_id}","starred_url":"https://api.github.com/users/shubhamdhama/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/shubhamdhama/subscriptions","organizations_url":"https://api.github.com/users/shubhamdhama/orgs","repos_url":"https://api.github.com/users/shubhamdhama/repos","events_url":"https://api.github.com/users/shubhamdhama/events{/privacy}","received_events_url":"https://api.github.com/users/shubhamdhama/received_events","type":"User","site_admin":false},"labels":[{"id":804160549,"node_id":"MDU6TGFiZWw4MDQxNjA1NDk=","url":"https://api.github.com/repos/zulip/zulip/labels/size:%20XL","name":"size: XL","color":"ededed","default":false}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":1,"created_at":"2018-06-06T16:56:08Z","updated_at":"2018-06-09T17:57:17Z","closed_at":null,"author_association":"MEMBER","pull_request":{"url":"https://api.github.com/repos/zulip/zulip/pulls/9696","html_url":"https://github.com/zulip/zulip/pull/9696","diff_url":"https://github.com/zulip/zulip/pull/9696.diff","patch_url":"https://github.com/zulip/zulip/pull/9696.patch"},"body":"Just trying this rule whether it's worth enabling or not.\r\n2nd and 3rd commit can be squashed if we decide to enable this feature.\r\n@timabbott FYI."},"comment":{"url":"https://api.github.com/repos/zulip/zulip/issues/comments/395987805","html_url":"https://github.com/zulip/zulip/pull/9696#issuecomment-395987805","issue_url":"https://api.github.com/repos/zulip/zulip/issues/9696","id":395987805,"node_id":"MDEyOklzc3VlQ29tbWVudDM5NTk4NzgwNQ==","user":{"login":"showell","id":142908,"node_id":"MDQ6VXNlcjE0MjkwOA==","avatar_url":"https://avatars2.githubusercontent.com/u/142908?v=4","gravatar_id":"","url":"https://api.github.com/users/showell","html_url":"https://github.com/showell","followers_url":"https://api.github.com/users/showell/followers","following_url":"https://api.github.com/users/showell/following{/other_user}","gists_url":"https://api.github.com/users/showell/gists{/gist_id}","starred_url":"https://api.github.com/users/showell/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/showell/subscriptions","organizations_url":"https://api.github.com/users/showell/orgs","repos_url":"https://api.github.com/users/showell/repos","events_url":"https://api.github.com/users/showell/events{/privacy}","received_events_url":"https://api.github.com/users/showell/received_events","type":"User","site_admin":false},"created_at":"2018-06-09T17:57:17Z","updated_at":"2018-06-09T17:57:17Z","author_association":"CONTRIBUTOR","body":"I think all of these lint rules make sense.  For the trailing-spaces thing, I think we definitely want it in eslint (I'd rather have a smart eslint config than relying on our custom linters), so then the question is whether the custom-lint check is easy to remove for just JS files.  And I'd be fine with the nuisance of double reporting--I feel like it happens elsewhere and it's pretty harmless."}}
{"actor":{"display_login":"showell"}}