It's funny that that's the answer that they always gave, considering there were times that we had screen shares, and I asked them to walk me through how they actually got it to work.

When they attempted to try to run it, unsurprisingly it broke.

There were even a few times that I didn't even review it and the first step I took was to inform them that it wouldn't run. Also, unsurprisingly, I was right.

Management at the time was driven by product development and delivery of "high-value" features. As long as deliverables were delivered, this dev could do anything they wanted to. At the end of a year, I'd lost about four weeks of productivity. That doesn't even cover the hours of after work time that I spent on trying to fix their fuckups.

Needless to say, I stopped doing that. I used to be a nice guy to work with, but now... Let's just say if you can't do the work, I'm not covering for you. If your PR doesn't get merged because it's broken and you can't fix it and you spend six weeks trying to fix it, that's on you.

[–] [email protected] 2 points 7 hours ago* (last edited 7 hours ago)

Oof.

My employer pays a buttload of money to CircleCI - for extensive checks (build, lint, formatting, full test suite, as well as custom scripts for translation converage, docs,... for the full tech stack) on every push. Reviews start only when everything passes.

I think you have given me a new-found appreciation for the reasoning behind that decision... 😄