Size: a a a

2021 March 02

A

Alexander in ctodailychat
Onlinehead
Кажется это стоит добавить в список аргументов про "вам точно не нужно N, все работает прекрасно и без него".
N8s
источник

O

Onlinehead in ctodailychat
Это слишком толстый намек)
источник

A

Alexander in ctodailychat
🙂
источник

A

Alexander in ctodailychat
Onlinehead
Это слишком толстый намек)
где N>=K
источник

SS

Slava Savitskiy in ctodailychat
Yaroslav
Смотря что вкладывается в понятие «плохой коммит»
ну после которого баг репорт
источник

SS

Slava Savitskiy in ctodailychat
при этом я не утверждаю, что мы все баги ловим на стадии ревью
источник

Y

Yaroslav in ctodailychat
Slava Savitskiy
при этом я не утверждаю, что мы все баги ловим на стадии ревью
Тогда я не понимаю в чем разница. Там ищете и тут ищете. Обмазываетесь фича тоглами и можете изолировать сайд эффекты. Архитектурно у вас должен быть предусмотрен паттерн стратегия, и по сути у вас с точки зрения багов которые нашли на ревью/тестировании/в проде процесс не меняется вообще никак
источник

SS

Slava Savitskiy in ctodailychat
Yaroslav
Тогда я не понимаю в чем разница. Там ищете и тут ищете. Обмазываетесь фича тоглами и можете изолировать сайд эффекты. Архитектурно у вас должен быть предусмотрен паттерн стратегия, и по сути у вас с точки зрения багов которые нашли на ревью/тестировании/в проде процесс не меняется вообще никак
искать в одном код ревью проще, чем когда у тебя с последнего известного хорошего состояния уже тыща коммитов
источник

Y

Yaroslav in ctodailychat
Правда то, о чем я говорю сдвигает баланс в сторону скорости разработки, а не качества. Иногда в таком процессе можно прилечь, если это для бизнеса не критично - то по сути мы с вами изобрели реализацию tbd
источник

Y

Yaroslav in ctodailychat
Slava Savitskiy
искать в одном код ревью проще, чем когда у тебя с последнего известного хорошего состояния уже тыща коммитов
Не понимаю о чем речь. Бисект будет работать так же, история коммитлв такая же
источник

SS

Slava Savitskiy in ctodailychat
так же как? если тебе, допустим, надо 15 минут, чтобы проверить один коммит, сколько времени ты будешь бисектить?
источник

O

Onlinehead in ctodailychat
Я так понимаю, что Ярослав постулирует то, что количество "плохих" коммитов будет небольшим и более высокие затраты на их исправление будут все равно выгоднее, чем проверка всего кода сразу. Но для меня это все очень уж перекликается с сознательным накоплением тех. долга, более того - даже форсирует само его возникновение.
источник

O

Onlinehead in ctodailychat
Но я бы тогда предложил и от тестов сразу отказаться. Вот они то точно кучу времени занимают, а разработчики вроде нормальные ребята, ошибаются редко, можно же тесты только на критичную бизнес логику писать...
источник

Y

Yaroslav in ctodailychat
Onlinehead
Я так понимаю, что Ярослав постулирует то, что количество "плохих" коммитов будет небольшим и более высокие затраты на их исправление будут все равно выгоднее, чем проверка всего кода сразу. Но для меня это все очень уж перекликается с сознательным накоплением тех. долга, более того - даже форсирует само его возникновение.
Да, это про копление техдолга в тех местах которые редко затрагиваются
источник

E

Eugene in ctodailychat
Да, + код ревью не только про “словить баг”
источник

O

Onlinehead in ctodailychat
Yaroslav
Да, это про копление техдолга в тех местах которые редко затрагиваются
А каким образом происходит разделение между редко и часто затрагиваемыми местами?
источник

SS

Slava Savitskiy in ctodailychat
Eugene
Да, + код ревью не только про “словить баг”
+
источник

Y

Yaroslav in ctodailychat
Slava Savitskiy
так же как? если тебе, допустим, надо 15 минут, чтобы проверить один коммит, сколько времени ты будешь бисектить?
Логорифм же:)
источник

O

Onlinehead in ctodailychat
Eugene
Да, + код ревью не только про “словить баг”
Эта аргументация уже приводилась:)
источник

Y

Yaroslav in ctodailychat
Onlinehead
А каким образом происходит разделение между редко и часто затрагиваемыми местами?
Эмпирически, разработчики не трогают код который не нужно делать в рамках задач
источник