{"action":"closed","number":3,"pull_request":{"url":"https://api.github.com/repos/BaterKari/reviewing-a-pull-request/pulls/3","id":233980557,"node_id":"MDExOlB1bGxSZXF1ZXN0MjMzOTgwNTU3","html_url":"https://github.com/BaterKari/reviewing-a-pull-request/pull/3","diff_url":"https://github.com/BaterKari/reviewing-a-pull-request/pull/3.diff","patch_url":"https://github.com/BaterKari/reviewing-a-pull-request/pull/3.patch","issue_url":"https://api.github.com/repos/BaterKari/reviewing-a-pull-request/issues/3","number":3,"state":"closed","locked":false,"title":"Change Tetris Colors","user":{"login":"github-learning-lab[bot]","id":37936606,"node_id":"MDM6Qm90Mzc5MzY2MDY=","avatar_url":"https://avatars0.githubusercontent.com/in/10572?v=4","gravatar_id":"","url":"https://api.github.com/users/github-learning-lab%5Bbot%5D","html_url":"https://github.com/apps/github-learning-lab","followers_url":"https://api.github.com/users/github-learning-lab%5Bbot%5D/followers","following_url":"https://api.github.com/users/github-learning-lab%5Bbot%5D/following{/other_user}","gists_url":"https://api.github.com/users/github-learning-lab%5Bbot%5D/gists{/gist_id}","starred_url":"https://api.github.com/users/github-learning-lab%5Bbot%5D/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/github-learning-lab%5Bbot%5D/subscriptions","organizations_url":"https://api.github.com/users/github-learning-lab%5Bbot%5D/orgs","repos_url":"https://api.github.com/users/github-learning-lab%5Bbot%5D/repos","events_url":"https://api.github.com/users/github-learning-lab%5Bbot%5D/events{/privacy}","received_events_url":"https://api.github.com/users/github-learning-lab%5Bbot%5D/received_events","type":"Bot","site_admin":false},"body":"# Using `request changes` on a pull request\n\nThanks again for all your help on that last pull request. With our powers combined, _anything_ is possible! This next change might introduce some new changes that aren't going to look great.\n\nSometimes, the author might identify that something isn't working. They might need a helping hand :wave: to solve the problem. In other instances, it might only work in their environment. When reviewing a pull request, look at and test the code to pinpoint any bugs or unexpected behavior.\n\n### Writing a review\n\n#### Review the `diff`\n\nReviewing the `diff`, the comparison of the proposed code, in the context of the whole project. Could it introduce performance problems, or security vulnerabilities? Try to predict unintended consequences that this change could cause.\n\n#### Try it out\n\nFor most things, actually trying out the proposed change is a good idea. This makes it a lot easier to tell if the actual change matches the intention. You can do this by:\n\n- Cloning the repository, and checking out to the branch compared in the pull request. Run the application in your local development environment.\n- Deploying the pull request to a review-lab or staging environment (as appropriate).\n\nWhen summarizing your review, let the author know if you completed any of these tests.\n\n#### Empathy and Constructive Feedback\n\nThe goal of providing feedback on a pull request is to ensure that the best possible change is being added. Sometimes, a change isn't addressing the problem in the best possible way. It's the reviewer's responsibility to provide meaningful and constructive feedback.\n\n### Requesting Changes 101\n\nBefore you submit your review, your line comments are pending and only visible to you. You can edit pending comments anytime before you submit your review. To cancel a pending review and its pending comments, scroll down to the end of the page in the Conversation tab. Then click **Cancel review**.\n\n## Step 4: Leave a review that asks for changes\n\nIf you could check out this code for me and tell me what is wrong, that would be fantastic.\n\n### :keyboard: Activity: Requesting changes\n\n1. On the pull request, click **Files changed**\n1. Hover over the line of code where you'd like to add a comment, and click the blue comment icon\n1. In the comment window, type your comment\n1. Click **Start a review**\n    - If you have already started a review, you can click **Add review comment**\n1. Above the changed code, click **Review changes**\n1. Type a comment summarizing your feedback on the proposed changes\n    - _Note: The problem here is that too many Tetris pieces are the same color. In your review, you might want to suggest that I choose colors which would be easier to see when playing the game._\n1. Select **Request changes** to submit feedback introducing changes necessary before merging\n1. Click **Submit review**\n\n<hr>\n<h3 align=\"center\">Return to this pull request for my next comment</h3>\n\n> _Sometimes I respond too fast for the page to update! If you perform an expected action and don't see a response, wait a few seconds and refresh the page for your next steps._\n","created_at":"2018-11-27T16:37:37Z","updated_at":"2018-11-27T17:14:34Z","closed_at":"2018-11-27T17:14:33Z","merged_at":"2018-11-27T17:14:33Z","merge_commit_sha":"1179ee291167ebabc66cc558a01c7440e6489e8b","assignee":{"login":"BaterKari","id":38873073,"node_id":"MDQ6VXNlcjM4ODczMDcz","avatar_url":"https://avatars3.githubusercontent.com/u/38873073?v=4","gravatar_id":"","url":"https://api.github.com/users/BaterKari","html_url":"https://github.com/BaterKari","followers_url":"https://api.github.com/users/BaterKari/followers","following_url":"https://api.github.com/users/BaterKari/following{/other_user}","gists_url":"https://api.github.com/users/BaterKari/gists{/gist_id}","starred_url":"https://api.github.com/users/BaterKari/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/BaterKari/subscriptions","organizations_url":"https://api.github.com/users/BaterKari/orgs","repos_url":"https://api.github.com/users/BaterKari/repos","events_url":"https://api.github.com/users/BaterKari/events{/privacy}","received_events_url":"https://api.github.com/users/BaterKari/received_events","type":"User","site_admin":false},"assignees":[{"login":"BaterKari","id":38873073,"node_id":"MDQ6VXNlcjM4ODczMDcz","avatar_url":"https://avatars3.githubusercontent.com/u/38873073?v=4","gravatar_id":"","url":"https://api.github.com/users/BaterKari","html_url":"https://github.com/BaterKari","followers_url":"https://api.github.com/users/BaterKari/followers","following_url":"https://api.github.com/users/BaterKari/following{/other_user}","gists_url":"https://api.github.com/users/BaterKari/gists{/gist_id}","starred_url":"https://api.github.com/users/BaterKari/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/BaterKari/subscriptions","organizations_url":"https://api.github.com/users/BaterKari/orgs","repos_url":"https://api.github.com/users/BaterKari/repos","events_url":"https://api.github.com/users/BaterKari/events{/privacy}","received_events_url":"https://api.github.com/users/BaterKari/received_events","type":"User","site_admin":false}],"requested_reviewers":[],"requested_teams":[],"labels":[],"milestone":null,"commits_url":"https://api.github.com/repos/BaterKari/reviewing-a-pull-request/pulls/3/commits","review_comments_url":"https://api.github.com/repos/BaterKari/reviewing-a-pull-request/pulls/3/comments","review_comment_url":"https://api.github.com/repos/BaterKari/reviewing-a-pull-request/pulls/comments{/number}","comments_url":"https://api.github.com/repos/BaterKari/reviewing-a-pull-request/issues/3/comments","statuses_url":"https://api.github.com/repos/BaterKari/reviewing-a-pull-request/statuses/92740017782eb730c821cda0172999cf30049ad5","head":{"label":"BaterKari:update-colors","ref":"update-colors","sha":"92740017782eb730c821cda0172999cf30049ad5","user":{"login":"BaterKari","id":38873073,"node_id":"MDQ6VXNlcjM4ODczMDcz","avatar_url":"https://avatars3.githubusercontent.com/u/38873073?v=4","gravatar_id":"","url":"https://api.github.com/users/BaterKari","html_url":"https://github.com/BaterKari","followers_url":"https://api.github.com/users/BaterKari/followers","following_url":"https://api.github.com/users/BaterKari/following{/other_user}","gists_url":"https://api.github.com/users/BaterKari/gists{/gist_id}","starred_url":"https://api.github.com/users/BaterKari/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/BaterKari/subscriptions","organizations_url":"https://api.github.com/users/BaterKari/orgs","repos_url":"https://api.github.com/users/BaterKari/repos","events_url":"https://api.github.com/users/BaterKari/events{/privacy}","received_events_url":"https://api.github.com/users/BaterKari/received_events","type":"User","site_admin":false},"repo":{"id":159239576,"node_id":"MDEwOlJlcG9zaXRvcnkxNTkyMzk1NzY=","name":"reviewing-a-pull-request","full_name":"BaterKari/reviewing-a-pull-request","private":false,"owner":{"login":"BaterKari","id":38873073,"node_id":"MDQ6VXNlcjM4ODczMDcz","avatar_url":"https://avatars3.githubusercontent.com/u/38873073?v=4","gravatar_id":"","url":"https://api.github.com/users/BaterKari","html_url":"https://github.com/BaterKari","followers_url":"https://api.github.com/users/BaterKari/followers","following_url":"https://api.github.com/users/BaterKari/following{/other_user}","gists_url":"https://api.github.com/users/BaterKari/gists{/gist_id}","starred_url":"https://api.github.com/users/BaterKari/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/BaterKari/subscriptions","organizations_url":"https://api.github.com/users/BaterKari/orgs","repos_url":"https://api.github.com/users/BaterKari/repos","events_url":"https://api.github.com/users/BaterKari/events{/privacy}","received_events_url":"https://api.github.com/users/BaterKari/received_events","type":"User","site_admin":false},"html_url":"https://github.com/BaterKari/reviewing-a-pull-request","description":null,"fork":false,"url":"https://api.github.com/repos/BaterKari/reviewing-a-pull-request","forks_url":"https://api.github.com/repos/BaterKari/reviewing-a-pull-request/forks","keys_url":"https://api.github.com/repos/BaterKari/reviewing-a-pull-request/keys{/key_id}","collaborators_url":"https://api.github.com/repos/BaterKari/reviewing-a-pull-request/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/BaterKari/reviewing-a-pull-request/teams","hooks_url":"https://api.github.com/repos/BaterKari/reviewing-a-pull-request/hooks","issue_events_url":"https://api.github.com/repos/BaterKari/reviewing-a-pull-request/issues/events{/number}","events_url":"https://api.github.com/repos/BaterKari/reviewing-a-pull-request/events","assignees_url":"https://api.github.com/repos/BaterKari/reviewing-a-pull-request/assignees{/user}","branches_url":"https://api.github.com/repos/BaterKari/reviewing-a-pull-request/branches{/branch}","tags_url":"https://api.github.com/repos/BaterKari/reviewing-a-pull-request/tags","blobs_url":"https://api.github.com/repos/BaterKari/reviewing-a-pull-request/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/BaterKari/reviewing-a-pull-request/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/BaterKari/reviewing-a-pull-request/git/refs{/sha}","trees_url":"https://api.github.com/repos/BaterKari/reviewing-a-pull-request/git/trees{/sha}","statuses_url":"https://api.github.com/repos/BaterKari/reviewing-a-pull-request/statuses/{sha}","languages_url":"https://api.github.com/repos/BaterKari/reviewing-a-pull-request/languages","stargazers_url":"https://api.github.com/repos/BaterKari/reviewing-a-pull-request/stargazers","contributors_url":"https://api.github.com/repos/BaterKari/reviewing-a-pull-request/contributors","subscribers_url":"https://api.github.com/repos/BaterKari/reviewing-a-pull-request/subscribers","subscription_url":"https://api.github.com/repos/BaterKari/reviewing-a-pull-request/subscription","commits_url":"https://api.github.com/repos/BaterKari/reviewing-a-pull-request/commits{/sha}","git_commits_url":"https://api.github.com/repos/BaterKari/reviewing-a-pull-request/git/commits{/sha}","comments_url":"https://api.github.com/repos/BaterKari/reviewing-a-pull-request/comments{/number}","issue_comment_url":"https://api.github.com/repos/BaterKari/reviewing-a-pull-request/issues/comments{/number}","contents_url":"https://api.github.com/repos/BaterKari/reviewing-a-pull-request/contents/{+path}","compare_url":"https://api.github.com/repos/BaterKari/reviewing-a-pull-request/compare/{base}...{head}","merges_url":"https://api.github.com/repos/BaterKari/reviewing-a-pull-request/merges","archive_url":"https://api.github.com/repos/BaterKari/reviewing-a-pull-request/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/BaterKari/reviewing-a-pull-request/downloads","issues_url":"https://api.github.com/repos/BaterKari/reviewing-a-pull-request/issues{/number}","pulls_url":"https://api.github.com/repos/BaterKari/reviewing-a-pull-request/pulls{/number}","milestones_url":"https://api.github.com/repos/BaterKari/reviewing-a-pull-request/milestones{/number}","notifications_url":"https://api.github.com/repos/BaterKari/reviewing-a-pull-request/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/BaterKari/reviewing-a-pull-request/labels{/name}","releases_url":"https://api.github.com/repos/BaterKari/reviewing-a-pull-request/releases{/id}","deployments_url":"https://api.github.com/repos/BaterKari/reviewing-a-pull-request/deployments","created_at":"2018-11-26T22:09:29Z","updated_at":"2018-11-27T16:37:38Z","pushed_at":"2018-11-27T17:14:33Z","git_url":"git://github.com/BaterKari/reviewing-a-pull-request.git","ssh_url":"git@github.com:BaterKari/reviewing-a-pull-request.git","clone_url":"https://github.com/BaterKari/reviewing-a-pull-request.git","svn_url":"https://github.com/BaterKari/reviewing-a-pull-request","homepage":"https://lab.github.com/githubtraining/reviewing-pull-requests","size":62,"stargazers_count":0,"watchers_count":0,"language":"HTML","has_issues":true,"has_projects":true,"has_downloads":true,"has_wiki":true,"has_pages":true,"forks_count":0,"mirror_url":null,"archived":false,"open_issues_count":0,"license":{"key":"mit","name":"MIT License","spdx_id":"MIT","url":"https://api.github.com/licenses/mit","node_id":"MDc6TGljZW5zZTEz"},"forks":0,"open_issues":0,"watchers":0,"default_branch":"master"}},"base":{"label":"BaterKari:master","ref":"master","sha":"eed4c1c6af7e3719ef7fc4253fb3682d979c477a","user":{"login":"BaterKari","id":38873073,"node_id":"MDQ6VXNlcjM4ODczMDcz","avatar_url":"https://avatars3.githubusercontent.com/u/38873073?v=4","gravatar_id":"","url":"https://api.github.com/users/BaterKari","html_url":"https://github.com/BaterKari","followers_url":"https://api.github.com/users/BaterKari/followers","following_url":"https://api.github.com/users/BaterKari/following{/other_user}","gists_url":"https://api.github.com/users/BaterKari/gists{/gist_id}","starred_url":"https://api.github.com/users/BaterKari/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/BaterKari/subscriptions","organizations_url":"https://api.github.com/users/BaterKari/orgs","repos_url":"https://api.github.com/users/BaterKari/repos","events_url":"https://api.github.com/users/BaterKari/events{/privacy}","received_events_url":"https://api.github.com/users/BaterKari/received_events","type":"User","site_admin":false},"repo":{"id":159239576,"node_id":"MDEwOlJlcG9zaXRvcnkxNTkyMzk1NzY=","name":"reviewing-a-pull-request","full_name":"BaterKari/reviewing-a-pull-request","private":false,"owner":{"login":"BaterKari","id":38873073,"node_id":"MDQ6VXNlcjM4ODczMDcz","avatar_url":"https://avatars3.githubusercontent.com/u/38873073?v=4","gravatar_id":"","url":"https://api.github.com/users/BaterKari","html_url":"https://github.com/BaterKari","followers_url":"https://api.github.com/users/BaterKari/followers","following_url":"https://api.github.com/users/BaterKari/following{/other_user}","gists_url":"https://api.github.com/users/BaterKari/gists{/gist_id}","starred_url":"https://api.github.com/users/BaterKari/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/BaterKari/subscriptions","organizations_url":"https://api.github.com/users/BaterKari/orgs","repos_url":"https://api.github.com/users/BaterKari/repos","events_url":"https://api.github.com/users/BaterKari/events{/privacy}","received_events_url":"https://api.github.com/users/BaterKari/received_events","type":"User","site_admin":false},"html_url":"https://github.com/BaterKari/reviewing-a-pull-request","description":null,"fork":false,"url":"https://api.github.com/repos/BaterKari/reviewing-a-pull-request","forks_url":"https://api.github.com/repos/BaterKari/reviewing-a-pull-request/forks","keys_url":"https://api.github.com/repos/BaterKari/reviewing-a-pull-request/keys{/key_id}","collaborators_url":"https://api.github.com/repos/BaterKari/reviewing-a-pull-request/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/BaterKari/reviewing-a-pull-request/teams","hooks_url":"https://api.github.com/repos/BaterKari/reviewing-a-pull-request/hooks","issue_events_url":"https://api.github.com/repos/BaterKari/reviewing-a-pull-request/issues/events{/number}","events_url":"https://api.github.com/repos/BaterKari/reviewing-a-pull-request/events","assignees_url":"https://api.github.com/repos/BaterKari/reviewing-a-pull-request/assignees{/user}","branches_url":"https://api.github.com/repos/BaterKari/reviewing-a-pull-request/branches{/branch}","tags_url":"https://api.github.com/repos/BaterKari/reviewing-a-pull-request/tags","blobs_url":"https://api.github.com/repos/BaterKari/reviewing-a-pull-request/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/BaterKari/reviewing-a-pull-request/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/BaterKari/reviewing-a-pull-request/git/refs{/sha}","trees_url":"https://api.github.com/repos/BaterKari/reviewing-a-pull-request/git/trees{/sha}","statuses_url":"https://api.github.com/repos/BaterKari/reviewing-a-pull-request/statuses/{sha}","languages_url":"https://api.github.com/repos/BaterKari/reviewing-a-pull-request/languages","stargazers_url":"https://api.github.com/repos/BaterKari/reviewing-a-pull-request/stargazers","contributors_url":"https://api.github.com/repos/BaterKari/reviewing-a-pull-request/contributors","subscribers_url":"https://api.github.com/repos/BaterKari/reviewing-a-pull-request/subscribers","subscription_url":"https://api.github.com/repos/BaterKari/reviewing-a-pull-request/subscription","commits_url":"https://api.github.com/repos/BaterKari/reviewing-a-pull-request/commits{/sha}","git_commits_url":"https://api.github.com/repos/BaterKari/reviewing-a-pull-request/git/commits{/sha}","comments_url":"https://api.github.com/repos/BaterKari/reviewing-a-pull-request/comments{/number}","issue_comment_url":"https://api.github.com/repos/BaterKari/reviewing-a-pull-request/issues/comments{/number}","contents_url":"https://api.github.com/repos/BaterKari/reviewing-a-pull-request/contents/{+path}","compare_url":"https://api.github.com/repos/BaterKari/reviewing-a-pull-request/compare/{base}...{head}","merges_url":"https://api.github.com/repos/BaterKari/reviewing-a-pull-request/merges","archive_url":"https://api.github.com/repos/BaterKari/reviewing-a-pull-request/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/BaterKari/reviewing-a-pull-request/downloads","issues_url":"https://api.github.com/repos/BaterKari/reviewing-a-pull-request/issues{/number}","pulls_url":"https://api.github.com/repos/BaterKari/reviewing-a-pull-request/pulls{/number}","milestones_url":"https://api.github.com/repos/BaterKari/reviewing-a-pull-request/milestones{/number}","notifications_url":"https://api.github.com/repos/BaterKari/reviewing-a-pull-request/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/BaterKari/reviewing-a-pull-request/labels{/name}","releases_url":"https://api.github.com/repos/BaterKari/reviewing-a-pull-request/releases{/id}","deployments_url":"https://api.github.com/repos/BaterKari/reviewing-a-pull-request/deployments","created_at":"2018-11-26T22:09:29Z","updated_at":"2018-11-27T16:37:38Z","pushed_at":"2018-11-27T17:14:33Z","git_url":"git://github.com/BaterKari/reviewing-a-pull-request.git","ssh_url":"git@github.com:BaterKari/reviewing-a-pull-request.git","clone_url":"https://github.com/BaterKari/reviewing-a-pull-request.git","svn_url":"https://github.com/BaterKari/reviewing-a-pull-request","homepage":"https://lab.github.com/githubtraining/reviewing-pull-requests","size":62,"stargazers_count":0,"watchers_count":0,"language":"HTML","has_issues":true,"has_projects":true,"has_downloads":true,"has_wiki":true,"has_pages":true,"forks_count":0,"mirror_url":null,"archived":false,"open_issues_count":0,"license":{"key":"mit","name":"MIT License","spdx_id":"MIT","url":"https://api.github.com/licenses/mit","node_id":"MDc6TGljZW5zZTEz"},"forks":0,"open_issues":0,"watchers":0,"default_branch":"master"}},"_links":{"self":{"href":"https://api.github.com/repos/BaterKari/reviewing-a-pull-request/pulls/3"},"html":{"href":"https://github.com/BaterKari/reviewing-a-pull-request/pull/3"},"issue":{"href":"https://api.github.com/repos/BaterKari/reviewing-a-pull-request/issues/3"},"comments":{"href":"https://api.github.com/repos/BaterKari/reviewing-a-pull-request/issues/3/comments"},"review_comments":{"href":"https://api.github.com/repos/BaterKari/reviewing-a-pull-request/pulls/3/comments"},"review_comment":{"href":"https://api.github.com/repos/BaterKari/reviewing-a-pull-request/pulls/comments{/number}"},"commits":{"href":"https://api.github.com/repos/BaterKari/reviewing-a-pull-request/pulls/3/commits"},"statuses":{"href":"https://api.github.com/repos/BaterKari/reviewing-a-pull-request/statuses/92740017782eb730c821cda0172999cf30049ad5"}},"author_association":"CONTRIBUTOR","merged":true,"mergeable":null,"rebaseable":null,"mergeable_state":"unknown","merged_by":{"login":"BaterKari","id":38873073,"node_id":"MDQ6VXNlcjM4ODczMDcz","avatar_url":"https://avatars3.githubusercontent.com/u/38873073?v=4","gravatar_id":"","url":"https://api.github.com/users/BaterKari","html_url":"https://github.com/BaterKari","followers_url":"https://api.github.com/users/BaterKari/followers","following_url":"https://api.github.com/users/BaterKari/following{/other_user}","gists_url":"https://api.github.com/users/BaterKari/gists{/gist_id}","starred_url":"https://api.github.com/users/BaterKari/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/BaterKari/subscriptions","organizations_url":"https://api.github.com/users/BaterKari/orgs","repos_url":"https://api.github.com/users/BaterKari/repos","events_url":"https://api.github.com/users/BaterKari/events{/privacy}","received_events_url":"https://api.github.com/users/BaterKari/received_events","type":"User","site_admin":false},"comments":2,"review_comments":1,"maintainer_can_modify":false,"commits":1,"additions":7,"deletions":7,"changed_files":1}}
{"actor":{"display_login":"BaterKari"}}