A walkthrough of a real code review process on a Java Google Photos clone project, covering how to review a community pull request. The process includes: checking out the PR locally, running tests first, avoiding bikeshedding on code style, and focusing on reducing lines of code. Key refactoring steps shown include simplifying JUnit parameterized tests by removing wrapper objects, reverting a version record back to a simpler enum with basic string matching, and restructuring a process-running utility method to return both exit code and output. The video also emphasizes the human/relationship side of code reviews, advocating for constructive, non-aggressive feedback.
•31m watch time
Sort: