Skip to main content
Home

Main navigation

  • About
    • Program at a glance
    • Venue and rooms
    • Lunch
    • Contribution
    • Accomodation
    • FAQs
    • In the Media
    • Team
  • Schedule
    • Sessions Overview
    • Mautic Days
    • Venue map
  • Speakers
  • Sponsors
  • Tracks
User account menu
  • Log in
Event Platform Header CTA Pictures

Breadcrumb

  1. Home
  2. A Drupal core maintainer shares peer code review best practices

A Drupal core maintainer shares peer code review best practices

Room
HS1 (Ground level)
Time Slot
Wed 4:15pm to 5:00pm (7/19/23)
Session length
Long session - 45min, including Q&A
Audience
Intermediate
Session Category
DevOps & Quality Assurance

 

Review session slides: Code Review Best Practices (xjm) -- Drupal Developer Days Vienna

Improve your teams' peer code review process, your code maintainability, and your Drupal contributions. Learn from a Drupal core release manager—with over ten years of Drupal core experience—how to give an efficient and effective peer code review.

Peer code reviews are essential, improving maintainability, increasing productivity, and reducing costs. They can also reduce the rate of defects (bugs) introduced into a codebase by as much as 80-90 percent. Most importantly, they build a shared understanding of the application within a team and are an opportunity for developers to mentor each other.

Drupal core's peer code review process has two phases:
1. An initial peer code review cycle with one or more other contributors
2. A final stage of code review by a Drupal core committer (one of the few like xjm who can accept changes into Drupal core's production codebase).

As a result, the core committers are exposed to all the peer review (good and not-so-good) throughout the Drupal core issue queue. This makes committers ideal mentors and subject matter experts on both best practices and pitfalls to avoid.

In this session, you will also discover research and psychological principles on when code review is effective and when it is not. You will learn how structuring your change sets for the reviewer can improve the whole team's code and maintainability. Finally, you will learn about automation and tooling that can help reviewers focus on what's most important.

Speaker(s)
Profile picture for user xjm

Jess (xjm)

Drupal core committer
Speaker biography

xjm is one of the Drupal core release managers and among the top contributors to Drupal 8, 9, and 10 issues. She is also a member of the Drupal Security Team and has helped coordinate most core security releases since Drupal 8's release in 2015. xjm is looking for funding

Session Keywords
Development process
Release processes
Back End Development
Contribution
Being Human
Share:

Platinum Sponsors

Logo 1xinternet

Gold Sponsors

D shaped logo and the name Droptica
Logo AgileDrop
Logo Amazee.io

Silver Sponsors

Logo Kraut.Hosting GmbH
dropsolid logo
Logo EOR Digital GmbH
Logo Druid.fi

Media Partner

Logo Kurier.at
droptimes logo

Funded by

wko logo
Meeting Destination Vienna

Footer

  • Contact
  • Code of Conduct
  • Data Privacy
  • Media Policy
  • FAQs
  • Imprint

Copyright © Drupal Dev Days 2025. All rights reserved.

Webdesign by acolono GmbH, implementation by Alex Milkovskyi

Webhosting by amazee.io
Powered by Drupal