I’m not going to lie. Remote is hard. Communicating is even harder when we can’t see each other’s facial expressions or hear the tone of voice.
I’ve received some feedback about tone in GitHub pull-requests that came off as less than friendly. It’s the first time I’ve seen this problem, and in the past, I’ve used a list I like to call the five commandments of code reviews.
“Just” is a poisoned word that tends to undermine one’s effort. If it were just this or that, you’d do it yourself.
We’ve all seen this. A piece of code gets introduced that, in your previous experience, caused problems. The gut reaction is to reject the PR with “this will break”. That tone of reply assumes the author hasn’t considered the issues and starts the conversation on the wrong tone. “Have you considered X?” is a much more productive way to address potential problems.
Most software engineers I know (myself included) have a huge ego problem. So, when someone tells us that the approach isn’t ideal, it stings. Remember that criticism is (and should be) about your work, and not directly about you.
Yeah, just no. Having a
:+1: buddy isn’t reviewing. It’s AaaS (Approvals as a Service).
There are so many missed opportunities when folks don’t spend 30m looking at unreviewed pull-requests. Getting more eyes on a problem means a better solution overall, and faster approvals mean faster shipping.
Remote is hard, but we’re Status; we’ve got this!