Code review: gatekeepers vs. coaches
I wrote a list of code review guidelines for the Servo project last year. They encapsulate the feedback that we expect our project contributors to provide when reviewing code changes, but I want to explore the motivation behind them. You’ll notice that almost every step is in the form “look for X”, followed by “suggest […]
Posted on February 7th, 2017 by Josh Matthews
Filed under: code, mozilla | No Comments »