CST438 Week 4

This week's reading from Software Engineering at Google was about code reviews. The part that stuck with me most was how Google splits the review across three roles: peer engineer, code base owner, and language readability approver. Each one is looking at the code differently, and that structure makes the whole process more intentional than what I'm used to.

What Stood Out

The bug fix review point was the most interesting. It's not enough to verify the fix works. The reviewer is also supposed to ask why the existing tests didn't catch the bug in the first place. That's a much higher bar, and a more useful one.

Takeaway

Reading about the code base owner role, someone asking whether the team can actually maintain this code if the original author leaves, made me realize that question doesn't get asked enough in most places.

Comments

Popular posts from this blog

CST363 Week 4 May 21st - May 27th

CST363 Week 3 May 14th - May 20th

CST489 Week 9