Ruby Rogues

216 RR Code Review Culture with Derek Prior


Listen Later

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 Questions12: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 Culture38:30 - Making Sure Code Reviews Get Done40: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.

Advertising Inquiries: https://redcircle.com/brands

Privacy & Opt-Out: https://redcircle.com/privacy

Become a supporter of this podcast: https://www.spreaker.com/podcast/ruby-rogues--6102073/support.
...more
View all episodesView all episodes
Download on the App Store

Ruby RoguesBy Charles M Wood

  • 4.5
  • 4.5
  • 4.5
  • 4.5
  • 4.5

4.5

45 ratings


More shows like Ruby Rogues

View all
The Changelog: Software Development, Open Source by Changelog Media

The Changelog: Software Development, Open Source

290 Listeners

The Ruby on Rails Podcast by Elise Shaffer

The Ruby on Rails Podcast

53 Listeners

Adventures in Angular by Charles M Wood

Adventures in Angular

33 Listeners

JavaScript Jabber by Charles M Wood

JavaScript Jabber

236 Listeners

iPhreaks by Charles M Wood

iPhreaks

17 Listeners

React Native Radio by Jamon Holmgren, Robin Heinze, Mazen Chami

React Native Radio

67 Listeners

Nerdland Podcast by Lieven Scheire

Nerdland Podcast

54 Listeners

Adventures in Angular by Charles M Wood

Adventures in Angular

15 Listeners

JavaScript Jabber by Charles M Wood

JavaScript Jabber

62 Listeners

Ruby Rogues by Charles M Wood

Ruby Rogues

21 Listeners

My Angular Story by Charles M Wood

My Angular Story

0 Listeners

My Ruby Story by Charles M Wood

My Ruby Story

0 Listeners

The Diary Of A CEO with Steven Bartlett by DOAC

The Diary Of A CEO with Steven Bartlett

8,361 Listeners

Remote Ruby by Chris Oliver, Andrew Mason

Remote Ruby

34 Listeners

Code with Jason by Jason Swett

Code with Jason

15 Listeners

Cautionary Tales with Tim Harford by Pushkin Industries

Cautionary Tales with Tim Harford

5,150 Listeners

The Rest Is History by Goalhanger

The Rest Is History

15,396 Listeners

The Ezra Klein Show by New York Times Opinion

The Ezra Klein Show

15,853 Listeners

The Rest Is Politics: Leading by Goalhanger

The Rest Is Politics: Leading

781 Listeners