
Five Reviewers Approved It. None of Them Were Wrong.
By the time I finished reading the pull request, it had already been merged. The author had asked me to review it. I was maybe forty minutes in, doing the slow thing: opening the files it touched, then opening the files it didn’t touch, trying to build a picture of what this change actually did to the app. Somewhere in the middle of that, two other engineers approved it, auto-merge fired, and eight commits squashed themselves onto master. ...