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, и в роли разработчика, который ждёт одобрения, пока его блестящий код устаревает. Неудобная правда заключается в том, что мы превратили ревью кода в подобие карго-культа. Мы проводим их, потому что «лучшие практики» говорят, что так надо, а не потому, что они действительно улучшают наш код или делают наши команды продуктивнее....
Picture this: you’re at your third daily standup this week, and someone inevitably asks, “So, what’s our velocity this sprint?” Meanwhile, your backlog has grown from a neat list of 20 items to a sprawling monster with 247 tickets, half of which haven’t been touched since the Obama administration. Sound familiar? Welcome to the dark side of Agile backlogs that nobody talks about at those fancy methodology conferences. Don’t get me wrong – I’m not here to burn down the Agile manifesto or suggest we all return to the waterfall days of our ancestors....