Skip to content

NaeemWillett-NOAA/skills-review-pull-requests

Folders and files

NameName
Last commit message
Last commit date

Latest commit

 

History

4 Commits
 
 
 
 
 
 
 
 
 
 

Repository files navigation

Review pull requests

Collaborate and work together on GitHub.

Step 3: Leave a review

You assigned yourself! 🎉

Pull request reviews ensure quality and maintain momentum of changes to your project.

When reviewing a pull request:

  1. Review the title and body of the pull request, and possibly any associated issue, to understand the intended change.
  2. Review the diff, the comparison of the proposed code, in the context of the whole project.
  3. For most things, try out the proposed change. Check if the actual change matches the intention. Find the repository's contributing guide to find out how to review the changes.

In your review comments:

  • Identify potential issues, risks, and limitations.
  • Suggest changes and improvements.
  • Share awareness of upcoming changes that the pull request doesn't account for.
  • Ask questions to verify shared understanding.
  • Highlight what the author did well and should keep doing.
  • Prioritize the most important feedback.
  • Be concise and provide meaningful detail.
  • Treat the pull request author with kindness and empathy.

When an approval or request for changes is not yet needed, consider using comments. An approval lets the author know you believe the pull request is safe to merge. Requesting changes lets the author know you believe the pull request is not ready to merge.

⌨️ Activity: Leave a review

  1. On the pull request, click Files changed.
  2. Click Review changes.
  3. Add a comment with your initial thoughts on the pull request.
  4. Select comment. You won't be able to approve or request changes to your own pull request.
  5. Click Submit review.
  6. Wait about 20 seconds then refresh this page (the one you're following instructions from). GitHub Actions will automatically update to the next step.

Get help: Post in our discussion boardReview the GitHub status page

© 2023 GitHub • Code of ConductMIT License

About

My clone repository

Resources

License

Stars

Watchers

Forks

Releases

No releases published

Packages

No packages published

Languages