Size: a a a

Боль Тимлида

2021 May 08

AK

Anton Kucherov in Боль Тимлида
Для замены код-ревью очевидно. :)
источник

СА

Сергей Аксёнов... in Боль Тимлида
Это был ответ не вам.
источник

VF

Victor Fabrichenko in Боль Тимлида
Сорян
источник

СА

Сергей Аксёнов... in Боль Тимлида
Вы противоречите сами себе. Все эти инструменты являются элементами обеспечения надёжности системы, и к принципу zero trust отношения не имеют, как и к понятию "команда".
источник

AB

Alexander B in Боль Тимлида
Что-то звучит как-будто ревью - это наказание какое )
источник

AD

Alexey Didukh in Боль Тимлида
Потому что тесты при рефакторинге/апдейтах помогут лучше любого код ревью. Ценность кодревью мимолётна и растворится завтра как дымка , а тесты и хорошая команда это отличная инвестиция
источник

AK

Anton Kucherov in Боль Тимлида
Никакой мониторинг не обеспечит надежность. Он только покажет где система оказалась ненадежной и обкакалась. 🙂
источник

C

Ctor in Боль Тимлида
И давайте тесты не будем ревьювить, зато полагаться на них будем.
источник

СА

Сергей Аксёнов... in Боль Тимлида
А код ревью полгода назад при развитии проекта поможет намного лучше, чем гипотетическая "хорошая команда" сегодня (что такое, кстати, "хорошая" команда, каковы критерии?)
источник

VF

Victor Fabrichenko in Боль Тимлида
Я знаю, что ИС меняется не сама по себе, а только при необходимости. Необходимость возникает там, где меняется мир, поскольку не весь мир меняется одинаково быстро, то и код в системе меняется в разных местах не одинаково часто. Если есть код, который написали и он в системе живёт годами, то нет никакого смысла его специально читать, если этот код спроектировали два человека. В других местах код меняется сильно чаще и его обязательно прочитают и скорее всего перепишут. Считать, что-то статистически не учитывая разную скорость изменения кода в разных частях системы, нуууу как бэ говорит об уровне понимания, по моему мнению
источник

СА

Сергей Аксёнов... in Боль Тимлида
Современные SRE оперируют метрикой "time to mitigate", а не "time without incidents".
источник

AS

Artem Shpynov in Боль Тимлида
Для этого есть линтеры
источник

AK

Anton Kucherov in Боль Тимлида
SRE не оперируют одной метрикой.
источник

VF

Victor Fabrichenko in Боль Тимлида
Просто самый крупный проект Сергея похоже 100-150 тыс строк, отсюда и соответствующее понимание. Хотя конечно, количество строк всего проекта особо не интересно, просто потому, что хорошо спроектированная ИС должна быть модульна и компоненты должны быть слабо связаны. В такой системе ее размеры не особо важны для любого программиста, он все равно в каждый момент времени видит только кусок. А если есть система которая требует знания "всей системы", то это лишь намекает на кривой дизайн
источник

AK

Anton Kucherov in Боль Тимлида
Блин я не думал что перерасти код-ревью реально настолько сложно и это настолько большое изменение в майндсете должно произойти.
источник

СА

Сергей Аксёнов... in Боль Тимлида
Я понимаю, что вам сложно аргументировать и очень хочется перейти на личности, но прошу вас этого не делать.

Действительно, те "горячие" части кодовой базы, которые часто меняются, как правило хорошо знакомы команде, но это не отменяет необходимости содержать в порядке всю кодовую базу, просто потому, что новые вводные от бизнеса могут поступить с любой стороны и в любой момент. Например: внедрение GDPR, и всей команде надо ознакомиться со всем неймспейсом, работающим с персданными, и дописать туда кучу функционала по архивации и удалению PD по запросу.
источник

VF

Victor Fabrichenko in Боль Тимлида
Вы же первый на личности перешли 😅
источник

СА

Сергей Аксёнов... in Боль Тимлида
Я этого и не утверждал.
источник

VF

Victor Fabrichenko in Боль Тимлида
Я вас просто отзеркалил, но при этом привел аргументацию, попробуйте ее опровергнуть и уже отвечайте на мой вопрос скорее
источник

AD

Alexey Didukh in Боль Тимлида
А я не знаю, что такое хорошая команда, у всех критерии разные. Для меня это коллеги, которые пишут код так, что я могу за вменяемое время в нем разобрался. А ещё они допускают ошибки, но быстро их исправляют, и больше не повторяют в будущем
источник