Devchat.tv Episode Roundup

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 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)

                              ...more
                              View all episodesView all episodes
                              Download on the App Store
                              Devchat.tv Episode RoundupBy Devchat.tv