Let me tell you about the most expensive “LGTM” I’ve ever seen. It was on a pull request that looked innocent enough – a small change to our payment processing logic. The reviewer, a senior engineer I respected, gave it a thumbs up with a comment that still haunts me: “Looks good! Nice work on keeping it simple 👍” That “simple” code went live and promptly charged 847 customers twice for their orders during Black Friday....
Позвольте мне рассказать о самом дорогом «LGTM», который я когда-либо видел. Он был в запросе на включение, который выглядел достаточно безобидно — небольшое изменение в нашей логике обработки платежей. Рецензент, старший инженер, которого я уважал, одобрил его, оставив комментарий, который до сих пор преследует меня: «Отлично выглядит! Хорошая работа, что всё просто 👍». Этот «простой» код был запущен и тут же дважды списал средства за заказы 847 клиентов во время Чёрной пятницы....
Let me start with a controversial statement that’ll probably ruffle some feathers: code reviews, as practiced by most teams, are a colossal waste of time. Before you sharpen your pitchforks and light your torches, hear me out. I’ve been on both sides of this fence – as a reviewer drowning in diffs and as a developer waiting for approval while my brilliant code grows stale. The uncomfortable truth is that we’ve turned code reviews into a cargo cult practice....
Начну с неоднозначного утверждения, которое, вероятно, вызовет возмущение: ревью кода, как их проводит большинство команд, — колоссальная трата времени. Прежде чем хвататься за вилы и факелы, выслушайте меня. Я был по обе стороны баррикад — и в роли ревьювера, погрязшего в diff, и в роли разработчика, который ждёт одобрения, пока его блестящий код устаревает. Неудобная правда заключается в том, что мы превратили ревью кода в подобие карго-культа. Мы проводим их, потому что «лучшие практики» говорят, что так надо, а не потому, что они действительно улучшают наш код или делают наши команды продуктивнее....
Let me start with a confession that might ruffle some feathers: your shiny new AI-powered code review tool is probably making your code worse, not better. I know, I know. You just dropped a few hundred dollars on that premium subscription, and now you’re getting dopamine hits every time it flags another “potential issue.” But here’s the uncomfortable truth that nobody in the DevTools marketing department wants you to hear: you’ve fallen for the automation fallacy, and it’s time for an intervention....