Code Review

December 10, 2018

Code Reviewer Spotlight: James Slater

PullRequest reviewer James Slater learned Google’s code review process by doing. Over several months of demonstrating his understanding of the style guide, code clarity, language mastery and best practices, he was finally granted JavaScript Readability at Google — an internal certification that shows you understand Google’s best practices and coding style for a given language.

images/james-slater-small.jpg

Completing the demanding readability process showed James’ commitment to improving code quality and mentoring others, and it also highlighted what he liked and disliked about the code review process. Over his engineering career, he found the right balance of positive, constructive guidance, and now provides code review as a service with PullRequest.

Brennan Angel headshot

December 4, 2018

Why Truly Great Product Managers Love Code Review

Many years ago, I started my career in product management with only a basic understanding of what is code review—developers looking through each others’ code for bugs. With upcoming deadlines, KPIs, user (re)engagement, compliance, and revenue goals all competing for my attention, understanding our development team’s code review process wasn’t a priority. After all, we had a great QA team. It took some time for me to understand that code review is so much more than identifying new issues.

Dan Mateer headshot

November 26, 2018

Code Reviewer Spotlight: Tristan Dunn

With experience scaling development teams at companies like Clearbit, Envoy, ROLI, Dribbble, and Google, PullRequest reviewer Tristan Dunn’s resume spans many different creative industries. Looking across them all and his work with PullRequest, he sees how new hires or new developers provide a fresh perspective through code review, and how sometimes that’s exactly what’s needed to find code smells that teams may have become nose blind to.

images/tristan-dunn-small.jpg
Our Code Reviewer Spotlight is an ongoing series of interviews with top reviewers to learn more about the senior developers in our reviewer network:

Brennan Angel headshot

November 14, 2018

Code Reviewer Spotlight: Fahad Zafar

Before working at Google and Amazon, Fahad Zafar points to code review from senior software engineers as the key tool that helped him grow as a developer. Now, as one of the thousands of reviewers at PullRequest, Fahad seeks to pay it forward by helping development teams improve code quality through code review.

images/fahad-zafar-small.jpg

While PullRequest reviewers come from diverse backgrounds, some common themes across reviewers include a passion for improving code quality, and a strong desire to help developers build better software. We interviewed our top reviewers and asked them the following seven questions:

Brennan Angel headshot

November 5, 2018

Code Reviewer Spotlight: Dylan Drop

A good code review process can be the fastest feedback loop for developers to receive actionable feedback from senior software engineers. PullRequest reviewer Dylan Drop also sees code review through the lens of mentorship, and emphasizes that developers and reviewers share a common goal of delivering high-quality code.

images/dylan-drop-small.jpg

Our Code Reviewer Spotlight is an ongoing series of interviews so you can get to know our top reviewers. Learn more about Dylan’s drive for continuous learning through code review below:

Brennan Angel headshot