{"action":"created","issue":{"url":"https://api.github.com/repos/tolujaiyeola/blog_application/issues/1","labels_url":"https://api.github.com/repos/tolujaiyeola/blog_application/issues/1/labels{/name}","comments_url":"https://api.github.com/repos/tolujaiyeola/blog_application/issues/1/comments","events_url":"https://api.github.com/repos/tolujaiyeola/blog_application/issues/1/events","html_url":"https://github.com/tolujaiyeola/blog_application/pull/1","id":70600671,"number":1,"title":"Peer review","user":{"login":"lamh85","id":2058381,"avatar_url":"https://avatars.githubusercontent.com/u/2058381?v=3","gravatar_id":"","url":"https://api.github.com/users/lamh85","html_url":"https://github.com/lamh85","followers_url":"https://api.github.com/users/lamh85/followers","following_url":"https://api.github.com/users/lamh85/following{/other_user}","gists_url":"https://api.github.com/users/lamh85/gists{/gist_id}","starred_url":"https://api.github.com/users/lamh85/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/lamh85/subscriptions","organizations_url":"https://api.github.com/users/lamh85/orgs","repos_url":"https://api.github.com/users/lamh85/repos","events_url":"https://api.github.com/users/lamh85/events{/privacy}","received_events_url":"https://api.github.com/users/lamh85/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2015-04-24T06:25:14Z","updated_at":"2015-04-24T20:19:17Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/tolujaiyeola/blog_application/pulls/1","html_url":"https://github.com/tolujaiyeola/blog_application/pull/1","diff_url":"https://github.com/tolujaiyeola/blog_application/pull/1.diff","patch_url":"https://github.com/tolujaiyeola/blog_application/pull/1.patch"},"body":"Here are all of the commits after your first one.\r\n\r\nThey include my comments. I reviewed the most recent few commits. I also reviewed parts that were causing errors: preventing the browser from loading the page."},"comment":{"url":"https://api.github.com/repos/tolujaiyeola/blog_application/issues/comments/96052565","html_url":"https://github.com/tolujaiyeola/blog_application/pull/1#issuecomment-96052565","issue_url":"https://api.github.com/repos/tolujaiyeola/blog_application/issues/1","id":96052565,"user":{"login":"gostevego","id":5523172,"avatar_url":"https://avatars.githubusercontent.com/u/5523172?v=3","gravatar_id":"","url":"https://api.github.com/users/gostevego","html_url":"https://github.com/gostevego","followers_url":"https://api.github.com/users/gostevego/followers","following_url":"https://api.github.com/users/gostevego/following{/other_user}","gists_url":"https://api.github.com/users/gostevego/gists{/gist_id}","starred_url":"https://api.github.com/users/gostevego/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/gostevego/subscriptions","organizations_url":"https://api.github.com/users/gostevego/orgs","repos_url":"https://api.github.com/users/gostevego/repos","events_url":"https://api.github.com/users/gostevego/events{/privacy}","received_events_url":"https://api.github.com/users/gostevego/received_events","type":"User","site_admin":false},"created_at":"2015-04-24T20:19:17Z","updated_at":"2015-04-24T20:19:17Z","body":"Overall, good job! I would also add a general comment, like this, that summarizes the important issues that need to be addressed (if any) such as the `,` in `<%= f.submit,:class => 'btn btn-primary' %>` before the pull request can be accepted. Creating Github issues that refer to these would be a good practice as well.\r\n\r\nIf you're working with a bigger team on a bigger project, it's highly likely that they'll follow code writing conventions. In that situation, you might be looking for cases where the code isn't written according to convention even though it might behave as desired. However, code reviewing practice can easily vary from team to team."}}