{"action":"created","issue":{"url":"https://api.github.com/repos/eslint/eslint/issues/4417","labels_url":"https://api.github.com/repos/eslint/eslint/issues/4417/labels{/name}","comments_url":"https://api.github.com/repos/eslint/eslint/issues/4417/comments","events_url":"https://api.github.com/repos/eslint/eslint/issues/4417/events","html_url":"https://github.com/eslint/eslint/issues/4417","id":116917053,"number":4417,"title":"no-arrow-condition does not recognise legitimate func","user":{"login":"nkbt","id":175264,"avatar_url":"https://avatars.githubusercontent.com/u/175264?v=3","gravatar_id":"","url":"https://api.github.com/users/nkbt","html_url":"https://github.com/nkbt","followers_url":"https://api.github.com/users/nkbt/followers","following_url":"https://api.github.com/users/nkbt/following{/other_user}","gists_url":"https://api.github.com/users/nkbt/gists{/gist_id}","starred_url":"https://api.github.com/users/nkbt/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/nkbt/subscriptions","organizations_url":"https://api.github.com/users/nkbt/orgs","repos_url":"https://api.github.com/users/nkbt/repos","events_url":"https://api.github.com/users/nkbt/events{/privacy}","received_events_url":"https://api.github.com/users/nkbt/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/eslint/eslint/labels/accepted","name":"accepted","color":"0052cc"},{"url":"https://api.github.com/repos/eslint/eslint/labels/bug","name":"bug","color":"fc2929"},{"url":"https://api.github.com/repos/eslint/eslint/labels/core","name":"core","color":"c7def8"}],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":14,"created_at":"2015-11-14T10:43:09Z","updated_at":"2015-11-17T22:07:06Z","closed_at":null,"body":"<img width=\"960\" alt=\"20151114-213400\" src=\"https://cloud.githubusercontent.com/assets/175264/11163110/a532c574-8b17-11e5-9681-692806f30050.png\">\r\n\r\nCode from https://github.com/nkbt/react-collapse/blob/master/src/Collapse.js#L45-L53:\r\n\r\n```js\r\n    return (\r\n      <Motion\r\n        defaultStyle={{height: 0}}\r\n        style={{height: spring(isOpened ? fixedHeight : 0, springConfig)}}>\r\n        {({height}) => (!isOpened && parseFloat(height).toFixed(1) === '0.0') ? null : (\r\n          <div style={{...style, height, overflow: 'hidden'}} {...props}>\r\n            {children}\r\n          </div>\r\n        )}\r\n      </Motion>\r\n    );\r\n```\r\n\r\n<bountysource-plugin>\r\n---\r\nWant to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/28244078-no-arrow-condition-does-not-recognise-legitimate-func?utm_campaign=plugin&utm_content=tracker%2F282608&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F282608&utm_medium=issues&utm_source=github).\r\n</bountysource-plugin>"},"comment":{"url":"https://api.github.com/repos/eslint/eslint/issues/comments/157524303","html_url":"https://github.com/eslint/eslint/issues/4417#issuecomment-157524303","issue_url":"https://api.github.com/repos/eslint/eslint/issues/4417","id":157524303,"user":{"login":"lukekarrys","id":542108,"avatar_url":"https://avatars.githubusercontent.com/u/542108?v=3","gravatar_id":"","url":"https://api.github.com/users/lukekarrys","html_url":"https://github.com/lukekarrys","followers_url":"https://api.github.com/users/lukekarrys/followers","following_url":"https://api.github.com/users/lukekarrys/following{/other_user}","gists_url":"https://api.github.com/users/lukekarrys/gists{/gist_id}","starred_url":"https://api.github.com/users/lukekarrys/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/lukekarrys/subscriptions","organizations_url":"https://api.github.com/users/lukekarrys/orgs","repos_url":"https://api.github.com/users/lukekarrys/repos","events_url":"https://api.github.com/users/lukekarrys/events{/privacy}","received_events_url":"https://api.github.com/users/lukekarrys/received_events","type":"User","site_admin":false},"created_at":"2015-11-17T22:07:06Z","updated_at":"2015-11-17T22:07:06Z","body":"> I'm wondering if we should have something like `no-confusing-arrows` that can catch strange cases like this anyway.\r\n\r\nI think that's a good idea. That rule would catch these cases and there'd be no need for an option since that's all the rule would cover. Then `no-constant-condition` would cover the rest, and `no-arrow-condition` could be deprecated."}}