Opinionated

Позвольте мне сказать вам кое-что, что может заставить вашу душу перфекциониста в области кодирования немного содрогнуться: технический долг не так страшен, как мы его себе представляем. На самом деле, я бы сказал, что относиться ко всему техническому долгу как к чему-то злому — это всё равно что утверждать, что все финансовые долги плохи. Попробуйте сказать это человеку, который только что купил дом своей мечты в ипотеку. Мы привыкли считать технический долг каким-то мрачным призраком, преследующим наши кодовые базы, чем-то, что нужно устранять любой ценой....

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, и в роли разработчика, который ждёт одобрения, пока его блестящий код устаревает. Неудобная правда заключается в том, что мы превратили ревью кода в подобие карго-культа. Мы проводим их, потому что «лучшие практики» говорят, что так надо, а не потому, что они действительно улучшают наш код или делают наши команды продуктивнее....