Code-Review

Иллюзия полезного робота-рецензента Вы знаете это чувство, когда кто-то говорит, что ваш код плох, но делает это самым профессиональным образом? Как будто получаете критику от робота в деловой одежде — нейтральный тон, отсутствие осуждения, максимальный эффект. Добро пожаловать в мир проверок кода с помощью ИИ, где нам как-то удалось сделать обратную связь одновременно менее эмоционально обременительной и бесконечно более напряжённой. Это аналог разработки программного обеспечения, когда вы обнаруживаете, что ваш партнёр по тренировкам теперь безжалостный надзиратель с ИИ, который никогда не устаёт....

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