{"action":"created","issue":{"url":"https://api.github.com/repos/peinanteng/leetcode/issues/38","repository_url":"https://api.github.com/repos/peinanteng/leetcode","labels_url":"https://api.github.com/repos/peinanteng/leetcode/issues/38/labels{/name}","comments_url":"https://api.github.com/repos/peinanteng/leetcode/issues/38/comments","events_url":"https://api.github.com/repos/peinanteng/leetcode/issues/38/events","html_url":"https://github.com/peinanteng/leetcode/issues/38","id":302104552,"number":38,"title":"It does not make sense to use ternary operator here","user":{"login":"nkukarl","id":8240960,"avatar_url":"https://avatars1.githubusercontent.com/u/8240960?v=4","gravatar_id":"","url":"https://api.github.com/users/nkukarl","html_url":"https://github.com/nkukarl","followers_url":"https://api.github.com/users/nkukarl/followers","following_url":"https://api.github.com/users/nkukarl/following{/other_user}","gists_url":"https://api.github.com/users/nkukarl/gists{/gist_id}","starred_url":"https://api.github.com/users/nkukarl/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/nkukarl/subscriptions","organizations_url":"https://api.github.com/users/nkukarl/orgs","repos_url":"https://api.github.com/users/nkukarl/repos","events_url":"https://api.github.com/users/nkukarl/events{/privacy}","received_events_url":"https://api.github.com/users/nkukarl/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":0,"created_at":"2018-03-04T17:34:09Z","updated_at":"2018-03-06T18:33:49Z","closed_at":"2018-03-06T18:33:49Z","author_association":"COLLABORATOR","body":"https://github.com/peinanteng/leetcode/blob/fe45b660dc2f699611d5ecd46b5a30b0fcecbf0a/containerWithMostWater.py#L12\r\n\r\nYou should do something like\r\n```\r\nif height[left] < height[right]:\r\n    left += 1\r\nelif height[left] > height[right]:\r\n    right -= 1\r\nelse:\r\n    left += 1\r\n    right -= 1\r\n```\r\nThen you don't need line 14.\r\nUsing ternary operator just makes the logic hard to follow."},"comment":{"url":"https://api.github.com/repos/peinanteng/leetcode/issues/comments/370882000","html_url":"https://github.com/peinanteng/leetcode/issues/38#issuecomment-370882000","issue_url":"https://api.github.com/repos/peinanteng/leetcode/issues/38","id":370882000,"user":{"login":"peinanteng","id":33048304,"avatar_url":"https://avatars1.githubusercontent.com/u/33048304?v=4","gravatar_id":"","url":"https://api.github.com/users/peinanteng","html_url":"https://github.com/peinanteng","followers_url":"https://api.github.com/users/peinanteng/followers","following_url":"https://api.github.com/users/peinanteng/following{/other_user}","gists_url":"https://api.github.com/users/peinanteng/gists{/gist_id}","starred_url":"https://api.github.com/users/peinanteng/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/peinanteng/subscriptions","organizations_url":"https://api.github.com/users/peinanteng/orgs","repos_url":"https://api.github.com/users/peinanteng/repos","events_url":"https://api.github.com/users/peinanteng/events{/privacy}","received_events_url":"https://api.github.com/users/peinanteng/received_events","type":"User","site_admin":false},"created_at":"2018-03-06T18:33:49Z","updated_at":"2018-03-06T18:33:49Z","author_association":"OWNER","body":"I prefer to keep concise code style."}}
{"actor":{"display_login":"peinanteng"}}