{"action":"opened","issue":{"url":"https://api.github.com/repos/RizOrt218/sorting-algorithms/issues/1","labels_url":"https://api.github.com/repos/RizOrt218/sorting-algorithms/issues/1/labels{/name}","comments_url":"https://api.github.com/repos/RizOrt218/sorting-algorithms/issues/1/comments","events_url":"https://api.github.com/repos/RizOrt218/sorting-algorithms/issues/1/events","html_url":"https://github.com/RizOrt218/sorting-algorithms/issues/1","id":126075023,"number":1,"title":"Code Review for `bubbleSort.js` file","user":{"login":"sgnl","id":3915598,"avatar_url":"https://avatars.githubusercontent.com/u/3915598?v=3","gravatar_id":"","url":"https://api.github.com/users/sgnl","html_url":"https://github.com/sgnl","followers_url":"https://api.github.com/users/sgnl/followers","following_url":"https://api.github.com/users/sgnl/following{/other_user}","gists_url":"https://api.github.com/users/sgnl/gists{/gist_id}","starred_url":"https://api.github.com/users/sgnl/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/sgnl/subscriptions","organizations_url":"https://api.github.com/users/sgnl/orgs","repos_url":"https://api.github.com/users/sgnl/repos","events_url":"https://api.github.com/users/sgnl/events{/privacy}","received_events_url":"https://api.github.com/users/sgnl/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":0,"created_at":"2016-01-12T01:15:40Z","updated_at":"2016-01-12T01:15:40Z","closed_at":null,"body":"# `bubbleSort.js`\r\nOverall this file is clean. consider adding a section to the `README.md` which explains what Bubble Sort is.\r\n\r\n**refactor**:\r\n[#1 Can you think of a way to refactor this code block to have just one `if` and `else` instead of having an `if` nested inside another `if`](https://github.com/RizOrt218/sorting-algorithms/blob/master/js/bubbleSort.js#L20)\r\n\r\n**Indentation:**\r\n[#1](https://github.com/RizOrt218/sorting-algorithms/blob/master/js/bubbleSort.js#L6)\r\n\r\n**clarify comments, consider using plain english**\r\n[#1](https://github.com/RizOrt218/sorting-algorithms/blob/master/js/bubbleSort.js#L12)\r\n\r\n**unneeded parens**:\r\n[#1](https://github.com/RizOrt218/sorting-algorithms/blob/master/js/bubbleSort.js#L20)"}}