{"action":"created","issue":{"url":"https://api.github.com/repos/mvila/reviewing-a-pull-request/issues/2","repository_url":"https://api.github.com/repos/mvila/reviewing-a-pull-request","labels_url":"https://api.github.com/repos/mvila/reviewing-a-pull-request/issues/2/labels{/name}","comments_url":"https://api.github.com/repos/mvila/reviewing-a-pull-request/issues/2/comments","events_url":"https://api.github.com/repos/mvila/reviewing-a-pull-request/issues/2/events","html_url":"https://github.com/mvila/reviewing-a-pull-request/pull/2","id":372259723,"node_id":"MDExOlB1bGxSZXF1ZXN0MjI0NTA1ODA3","number":2,"title":"Update Game Instructions","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},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":0,"created_at":"2018-10-20T22:43:36Z","updated_at":"2018-10-20T22:48:49Z","closed_at":null,"author_association":"NONE","pull_request":{"url":"https://api.github.com/repos/mvila/reviewing-a-pull-request/pulls/2","html_url":"https://github.com/mvila/reviewing-a-pull-request/pull/2","diff_url":"https://github.com/mvila/reviewing-a-pull-request/pull/2.diff","patch_url":"https://github.com/mvila/reviewing-a-pull-request/pull/2.patch"},"body":"# Commenting on Pull Requests\n\nAs a collaborator on a repository, you will get and give pull request reviews before merging code. This is important! It ensures quality code and maintains momentum of changes to your project.\n\nAs you can see, I've opened this pull request and requested you as a reviewer. Pull requests are places to _share_ the changes made to the repository.\n\n### Responsibilities of a reviewer\n\nAs a pull request reviewer, your role is to help the pull request author (thats me!) by making sure:\n\n- Code destined for production is of the highest quality\n- Intentional shortcuts (technical debt) are commented on and confirmed\n- The greater team is aware of potential risks associated with changes\n\nThese broad responsibilities can include more specific goals, like:\n\n- Pointing out potential issues in code quality, security, or business logic\n- Suggesting other reviewers to the author, when warranted\n- Commenting on, approving, or requesting changes on the PR\n- Providing suggestions for alternate or better implementation details\n\nSome repositories use a [CODEOWNERS](https://help.github.com/articles/about-codeowners/) file. The `CODEOWNERS` file assigns responsibility for certain parts of the code to specific individuals or teams. When the `CODEOWNERS` feature is enabled, only the code owner has the final authority to approve the pull request. Your review may not be a formal approval, but it does show confidence.\n\n## Step 2: Communicate in pull requests\n\nWhen an approval or request for changes is not yet needed, consider using **comments**. Comments enable you to inquire about the proposed change early in the review process. You don't need to wait until something is complete to make suggestions.\n\n### :keyboard: Activity: Comment on my pull request\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 a question or leave a more general comment\n1. When you're done, click **Add single comment**\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"},"comment":{"url":"https://api.github.com/repos/mvila/reviewing-a-pull-request/issues/comments/431623984","html_url":"https://github.com/mvila/reviewing-a-pull-request/pull/2#issuecomment-431623984","issue_url":"https://api.github.com/repos/mvila/reviewing-a-pull-request/issues/2","id":431623984,"node_id":"MDEyOklzc3VlQ29tbWVudDQzMTYyMzk4NA==","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},"created_at":"2018-10-20T22:48:49Z","updated_at":"2018-10-20T22:48:49Z","author_association":"NONE","body":"## Reviewing Pull Requests\n\nNice work @mvila. By providing feedback, you've enabled me to reevaluate how I am making my desired change. But, pull request reviews don't stop at commenting. You can also `approve` or `request changes` on my pull request.\n\nWhen you get asked to review something, you should acknowledge the request. Acknowledging a review request could take several forms. It could be leaving a comment on the pull request or leaving a review. In some cases, it could be rejecting the review request altogether.\n\n<details><summary>Before you review, some things to consider</summary>\n\n#### Discern the context\n\nReview the title and body of the pull request to understand the intended change. This should help you identify limitations, boundaries, and other important context.\n\n#### Observing the progress\n\nAs a reviewer, there are certain things to look for when identifying how to best provide feedback. In early stages, reviews should focus on the general direction of the changes. Identify if the goal is possible, rather than nitpicking the style, polish, or wording. Pull requests that are closer to merging should receive a robust review. Thorough testing is one of many ways to ensure the changes won't break the project.\n\nRegardless of timing, focus your feedback on the most essential changes. Suggest changes for minor issues within the pull request. When suggesting major changes, open a separate pull request against the author's branch.\n\n</details>\n\n## Step 3: Leave a review\n\nI've made some changes to this branch and would love :heart: it if you could `approve` this pull request. For now, just approving the pull request will suffice. In a moment we will look at requesting changes. :+1:\n\n### :keyboard: Activity: Approve a pull request\n\n1. In the pull request, click the **Files changed** tab\n1. Review the changes in the pull request by commenting on specific lines\n1. Above the changed code, click **Review changes**\n1. Type a comment summarizing your feedback on the proposed changes\n1. Select **Approve** to approve merging the changes proposed in the pull request\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"}}
{"actor":{"display_login":"github-learning-lab"}}