Exploring The Process And Practice Of Building Better Software Through Code Reviews

The Python Podcast.__init__

Episode | Podcast

Date: Sun, 04 Sep 2022 21:00:00 -0400

<div class="wp-block-jetpack-markdown"><h2>Summary</h2> <p>Writing code is only one piece of creating good software. Code reviews are an important step in the process of building applications that are maintainable and sustainable. In this episode On Freund shares his thoughts on the myriad purposes that code reviews serve, as well as exploring some of the patterns and anti-patterns that grow up around a seemingly simple process.</p> <h2>Announcements</h2> <ul> <li>Hello and welcome to Podcast.__init__, the podcast about Python&#8217;s role in data and science.</li> <li>When you&#8217;re ready to launch your next app or want to try a project you hear about on the show, you&#8217;ll need somewhere to deploy it, so take a look at our friends over at Linode. With their managed Kubernetes platform it&#8217;s easy to get started with the next generation of deployment and scaling, powered by the battle tested Linode platform, including simple pricing, node balancers, 40Gbit networking, dedicated CPU and GPU instances, and worldwide data centers. And now you can launch a managed MySQL, Postgres, or Mongo database cluster in minutes to keep your critical data safe with automated backups and failover. Go to <a href="https://www.pythonpodcast.com/linode?utm_source=rss&amp;utm_medium=rss">pythonpodcast.com/linode</a> and get a $100 credit to try out a Kubernetes cluster of your own. And don&#8217;t forget to thank them for their continued support of this show!</li> <li>Your host as usual is Tobias Macey and today I&#8217;m interviewing On Freund about the intricacies and importance of code reviews</li> </ul> <h2>Interview</h2> <ul> <li>Introductions</li> <li>How did you get introduced to Python?</li> <li>Can you start by giving us your description of what a code review is? <ul> <li>What is the purpose of the code review?</li> </ul> </li> <li>At face value a code review appears to be a simple task. What are some of the subtleties that become evident with time and experience?</li> <li>What are some of the ways that code reviews can go wrong?</li> <li>What are some common anti-patterns that get applied to code reviews?</li> <li>What are the elements of code review that are useful to automate? <ul> <li>What are some of the risks/bad habits that can result from overdoing automated checks/fixes or over-reliance on those tools in code reviews?</li> </ul> </li> <li>identifying who can/should do a review for a piece of code</li> <li>how to use code reviews as a teaching tool for new/junior engineers</li> <li>how to use code reviews for avoiding siloed experience/promoting cross-training</li> <li>PR templates for capturing relevant context</li> <li>What are the most interesting, innovative, or unexpected ways that you have seen code reviews used?</li> <li>What are the most interesting, unexpected, or challenging lessons that you have learned while leading and supporting engineering teams?</li> <li>What are some resources that you recommend for anyone who wants to learn more about code review strategies and how to use them to scale their teams?</li> </ul> <h2>Keep In Touch</h2> <ul> <li><a href="https://www.linkedin.com/in/onfreund/?utm_source=rss&amp;utm_medium=rss" rel="noopener" target="_blank">LinkedIn</a></li> <li><a href="https://twitter.com/onfreund?lang=en&amp;utm_source=rss&amp;utm_medium=rss" rel="noopener" target="_blank">@onfreund</a> on Twitter</li> </ul> <h2>Picks</h2> <ul> <li>Tobias <ul> <li><a href="https://www.goodreads.com/book/show/28110852-the-girl-who-drank-the-moon?utm_source=rss&amp;utm_medium=rss" rel="noopener" target="_blank">The Girl Who Drank The Moon</a></li> </ul> </li> <li>On <ul> <li><a href="https://www.imdb.com/title/tt3032476/?utm_source=rss&amp;utm_medium=rss" rel="noopener" target="_blank">Better Call Saul</a></li> </ul> </li> </ul> <h2>Closing Announcements</h2> <ul> <li>Thank you for listening! Don&#8217;t forget to check out our other shows. The <a href="https://www.dataengineeringpodcast.com?utm_source=rss&amp;utm_medium=rss" rel="noopener" target="_blank">Data Engineering Podcast</a> covers the latest on modern data management. The <a href="https://www.themachinelearningpodcast.com?utm_source=rss&amp;utm_medium=rss" rel="noopener" target="_blank">Machine Learning Podcast</a> helps you go from idea to production with machine learning.</li> <li>Visit the <a href="https://www.pythonpodcast.com?utm_source=rss&amp;utm_medium=rss">site</a> to subscribe to the show, sign up for the mailing list, and read the show notes.</li> <li>If you&#8217;ve learned something or tried out a project from the show then tell us about it! Email <a href="mailto:hosts@podcastinit.com">hosts@podcastinit.com</a>) with your story.</li> <li>To help other people find the show please leave a review on <a href="https://itunes.apple.com/us/podcast/podcast.-init/id981834425?mt=2&amp;uo=6&amp;at=&amp;ct=&amp;utm_source=rss&amp;utm_medium=rss" rel="noopener" target="_blank">iTunes</a> and tell your friends and co-workers</li> </ul> <h2>Links</h2> <ul> <li><a href="https://www.trywilco.com/?utm_source=rss&amp;utm_medium=rss" rel="noopener" target="_blank">Wilco</a></li> <li><a href="https://en.wikipedia.org/wiki/Code_review?utm_source=rss&amp;utm_medium=rss" rel="noopener" target="_blank">Code Review</a></li> <li><a href="https://www.home-assistant.io/?utm_source=rss&amp;utm_medium=rss" rel="noopener" target="_blank">Home Assistant</a> <ul> <li><a href="https://www.pythonpodcast.com/episode-94-home-assistant-with-paulus-schoutsen/?utm_source=rss&amp;utm_medium=rss">Podcast Episode</a></li> </ul> </li> <li><a href="https://trunkbaseddevelopment.com/?utm_source=rss&amp;utm_medium=rss" rel="noopener" target="_blank">Trunk-based Development</a></li> <li><a href="https://trunkbaseddevelopment.com/?utm_source=rss&amp;utm_medium=rss" rel="noopener" target="_blank">Git Flow</a></li> <li><a href="https://nvie.com/posts/a-successful-git-branching-model/?utm_source=rss&amp;utm_medium=rss" rel="noopener" target="_blank">Pair Programming</a></li> <li><a href="https://martinfowler.com/articles/feature-toggles.html?utm_source=rss&amp;utm_medium=rss" rel="noopener" target="_blank">Feature Flags</a> <ul> <li><a href="https://www.pythonpodcast.com/feature-flags-episode-239/?utm_source=rss&amp;utm_medium=rss">Podcast Episode</a></li> </ul> </li> <li><a href="https://en.wikipedia.org/wiki/Performance_indicator?utm_source=rss&amp;utm_medium=rss" rel="noopener" target="_blank">KPI == Key Performance Indicator</a></li> <li><a href="https://mitodl.github.io/handbook/?utm_source=rss&amp;utm_medium=rss" rel="noopener" target="_blank">MIT Open Learning Engineering Handbook</a></li> <li><a href="https://github.com/python/peps?utm_source=rss&amp;utm_medium=rss" rel="noopener" target="_blank">PEP Repository</a></li> </ul> <p>The intro and outro music is from Requiem for a Fish <a href="http://freemusicarchive.org/music/The_Freak_Fandango_Orchestra/?utm_source=rss&amp;utm_medium=rss" rel="noopener" target="_blank">The Freak Fandango Orchestra</a> / <a href="http://creativecommons.org/licenses/by-sa/3.0/?utm_source=rss&amp;utm_medium=rss" rel="noopener" target="_blank">CC BY-SA</a></p> </div> <img alt="" height="0" src="https://analytics.boundlessnotions.com/piwik.php?idsite=1&amp;rec=1&amp;url=https%3A%2F%2Fwww.pythonpodcast.com%2Fcode-review-for-better-software-episode-376%2F&amp;action_name=Exploring+The+Process+And+Practice+Of+Building+Better+Software+Through+Code+Reviews+-+Episode+376&amp;urlref=https%3A%2F%2Fwww.pythonpodcast.com%2Ffeed%2F&amp;utm_source=rss&amp;utm_medium=rss" style="border: 0; width: 0; height: 0;" width="0" />