code-review

tidyverse/code-review

code-review.tidyverse.org

34 stars

9 forks

Best Practices Data Wrangling Machine Learning

The Tidyteam Code Review Principles guide provides a comprehensive framework for conducting effective code reviews across Posit’s open-source ecosystems, including tidyverse, tidymodels, and mlops. Designed as both a learning resource for new contributors and a reference for experienced developers, this guide establishes consistent standards for collaborative software development. Whether you’re reviewing pull requests or submitting your own contributions, these principles help teams maintain high-quality code while fostering a constructive and inclusive development culture.

The guide covers essential aspects of the code review workflow through nine core principles, including how to evaluate pull requests effectively, provide actionable feedback, manage review timelines, and resolve technical disagreements. It offers practical guidance for both reviewers and authors, from crafting focused pull requests to responding thoughtfully to feedback. With GitHub-specific best practices and integration with R development tools like the usethis::pr_*() functions, this resource streamlines the review process and helps data scientists and developers collaborate more efficiently on shared codebases.

Contributors#

Resources featuring code-review#

Helping developers thrive | Cat Hicks | Data Science Hangout

To join future data science hangouts, add it to your calendar here: https://pos.it/dsh - All are welcome! We’d love to see you! We were recently joined by Dr. Cat Hicks, Psychologist for software teams at Catharsis Consulting and upcoming positconf 2025 keynote speaker, to chat about the psychology of software teams, organizational culture and team dynamics, debunking tech industry myths, and strategies for professional visibility. Cat’s work is in applied social science, bringing evidence-based answers to real-world workplace questions, especially for software development teams. She’s not a “couch psychologist” but rather focuses on “structures of beliefs and behavioral science and team practices”. Here are some of the themes Cat touches on in the conversation: Understanding developer (and Data Scientist!) thriving Debunking industry myths, especially the “10x Developer” myth “Designing an organization that wants to understand itself” Fostering psychological safety in remote, multi-disciplinary teams Getting recognized for work by being strategically performative Cat has a unique perspective that bridges psychology and the practical realities of tech and data roles. She’s got our backs! Resources mentioned in the video and zoom chat: Cat Hicks’s newsletter, “Fight for the Human” → https://www.fightforthehuman.com/ Cat Hicks’s podcast, “Change, Technically” (specifically the episode “You Deserve Better Brain Research”) → https://www.changetechnically.fyi/2396236/episodes/17378968-you-deserve-better-brain-research Cat Hicks’s Bluesky profile → https://bsky.app/profile/grimalkina.bsky.social Developer Thriving paper (AI skill threat paper) → https://osf.io/preprints/psyarxiv/2gej5_v2 Code Review Anxiety paper: “Understanding and effectively mitigating code review anxiety” → https://link.springer.com/article/10.1007/s10664-024-10550-9 Code Review Anxiety workbook → https://developer-success-lab.gitbook.io/code-review-anxiety-workbook-1 Ted Laderas’s post on how improvisational comedy rules can help collaborations → https://laderast.github.io/posts/2022-02-27-yes-and-is-the-foundation-of-collaboration/ Posit Conference information → https://posit.co/conference/ useR 2025 conference information → https://user2025.r-project.org ► Subscribe to Our Channel Here: https://bit.ly/2TzgcOu Follow Us Here: Website: https://www.posit.co Hangout: https://pos.it/dsh LinkedIn: https://www.linkedin.com/company/posit-software Bluesky: https://bsky.app/profile/posit.co Thanks for hanging out with us! Timestamps: 00:00 Intro 01:11 Cat Hicks Introduces herself 04:04 What does Catharsis Consulting help with? 07:45 Preview of Cat Hicks’s Posit conference keynote 10:14 Tips on how to objectively assess team comfort with change 19:14 How much of developer thriving comes from your immediate team vs. the larger organization? 23:56 Strategies to foster psychological safety and empathy in remote, multi-disciplinary teams 29:04 Have you had instances where you felt the need to correct misconceptions about a paper? (the myth of “10x developers” or “10x engineers”) 34:34 Have you looked at interventions that explicitly scaffold information exchange in groups? 40:48 Ways organizations can balance top-down and bottom-up approaches to culture 47:19 What can staff with less formal power do to sow positive seeds in a low-morale culture? 52:00 How to get noticed for long-term, high-impact work without appearing unproductive or overly performative

Empowering individuals through AI tools, technologies, and custom apps | Chris Engelhardt @ Gen Re

“Empathy, I think, is a hallmark of a well functioning team.” Chris Engelhardt might lead data teams and AI initiatives, but he’s also got a PhD in Social Psychology and a keen interest in helping people work more happily. This episode of the Hangout went to amazing and unexpected places as we talked about AI use cases, getting buy-in from leadership, and building stakeholder trust alongside building psychological safety in your teams, code review anxiety, and team wellbeing. If you’re curious about enterprise level AI use cases, or how Posit integrates with Databricks, there’s lots to learn from Chris! We hope you’ll join us live soon! Timestamps 02:35 About Chris Engelhardt Sr. Data and AI Operations Manager at Gen Re 05:55 How do you get people engaged in a vision for AI? 08:10 What are some use cases for AI? 11:40 How do you manage international data science teams? 13:15 What one AI skill data analysts should learn? 14:50 What people skills should data analysts and data scientists learn? 16:25 How do you get buy-in from leadership and data scientists for AI initiatives? 18:25 What is AI? Where do I start? What are pitfalls of AI to avoid? 22:40 Integrating Posit with Databricks, Unity Catalog, and pins 25:55 How do you handle it when two different teams manage Posit versus Databricks? 27:30 What if AI tools provide inaccurate results? How do you regain trust with stakeholders? 29:50 How do you protect your own and your data team’s wellbeing at work? 34:20 How has AI changed hiring and job descriptions? Has AI changed data roles? 36:30 Code review anxiety and psychological safety around code critiques and pull requests 40:00 Helping your team feel safe and willing to talk about things and collaborate 41:40 How do you encourage psychological safety in a cross-cultural international team? 47:15 How do you use Posit pins with Databricks? 49:50 Career advice for working in data - you can’t control everything, but you control your effort 51:15 Don’t underestimate the value of connecting with others 53:33 Does using pins with Databricks Unity Catalog present a challenge with data governance? 56:45 How do you encourage learning and upskilling within your team? Resources mentioned in this episode: Code Review Anxiety Toolkit: https://developer-success-lab.gitbook.io/code-review-anxiety-workbook-1/part-three-mini-code-review-anxiety-toolkit/introduction Matthew Montero from Gen Re’s episode: https://posit.co/data-science-hangout/56-matthew-montero/ Schedule time to chat about Databricks and Posit here: https://posit.co/schedule-a-call/?booking_calendar__c=Databricks Posit Team Workflow Demos: https://pos.it/team-demo The Psychological Safety Handbook Daren mentioned: https://handbook.gitlab.com/handbook/leadership/emotional-intelligence/psychological-safety/