Episode 222: 216 RR Code Review Culture with Derek Prior

Ruby Rogues

English - July 15, 2015 15:00 - 1 hour - 60.1 MB - ★★★★★ - 44 ratings
How To Education News Tech News ruby ruby on rails rails programming technology developer development careers Homepage Download Apple Podcasts Google Podcasts Overcast Castro Pocket Casts RSS feed

Previous Episode: 215 RR Sonic Pi with Sam Aaron

Next Episode: 217 RR Architecture Without an End State with Michael Nygard

03:42 - Derek Prior Introduction

Twitter
GitHub
Blog
thoughtbot

@thoughtbot
thoughtbot Code Review Guides

The Bike Shed Podcast

@_bikeshed

04:01 - Code Reviews

Derek Prior: Implementing a Strong Code-Review Culture @ RailsConf 2015

Slides

05:14 - What happens when you don’t do code reviews?

06:30 - Not Emphasizing Code Quality, Setting Code Review Up for Failure

Edge Cases
Diverse Feedback, Team Conflict

10:43 - Code Reviewing Yourself: Answering Your Own Questions

12:03 - The Evolution of Code Review (Code Review as an Asynchronous Process)

14:51 - Small Changes, “Pull Request Bombs”

Handling Architectural Disagreements and Discussions
Improving the Design of Existing Code by Martin Fowler (with Kent Beck, John Brant, William Opdyke, and Don Roberts)

23:49 - Making Code Review a Supportive Process

Stop Issuing Commands; Ask Probing Questions
DON’T Use “Why didn’t you ________?”
DO Use “Have you considered _________?”
or “That’s interesting…I might have used _______.”

30:32 - What qualities should reviewees have?

34:27 - Getting Code Reviews Introduced Into Company Culture

38:30 - Making Sure Code Reviews Get Done

40:47 - Tagging Specific Team Members

LGTM = Looks Good To Me
Gerrit

44:39 - Other Handy Code Review Tools

Style Guides

rubocop
JSHint
sass-lint

Hound

repo

Code Climate

47:49 - Code Review Feedback Resources for Solo Programmers

exercism.io
pairprogramwith.me
CodeNewbie

Ruby Monday
JavaScript Tuesday
Python Thursday

Picks

Code Newbie Podcast: Sandi Metz Part I (Saron)

Code Newbie Podcast: Sandi Metz Part II (Saron)

If Google Were A Guy (Saron)

LEGO Ideas - Lovelace & Babbage (Coraline)

CoverMyMeds is offering Ruby on Rails training for experienced developers (David)

CoverMyMeds Billboard 1 (David)

CoverMyMeds Billboard 2 (David)

The Bike Shed Podcast (Derek)

The Ember RFC Process (Derek)

tota11y (Derek)

Eileen Uchitelle: How to Performance @ GoRuCo 2015 (Derek)

Olympus SP-100EE (Avdi)

Special Guest: Derek Prior.