Opinionated

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

Your code style guide might be the reason your team dreads Monday morning code reviews. If every pull request turns into a battlefield over semicolons and indentation, you’ve probably crossed the line from helpful guidance into micromanagement territory. Don’t get me wrong – style guides are essential. They’re the difference between a codebase that feels like a well-orchestrated symphony and one that sounds like a middle school band tuning their instruments....

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