В рейтинге участвует:

групп:

каналов:

Виртуальный сервер на SSD - недорого!

Аренда выделенных и виртуальных серверов (VDS/VPS), хостинг, аренда IP-адресов, администрирование, круглосуточная поддержка

qwarta.ru подробнее

Резервное копирование с проверкой на вирусы!!!

Удобный сервис создания резервных копий на любой сервер сети интернет. Отслеживайте изменения, проверяйте на вирусы. Надежно защитите свой бизнес!

go.backupland.com

Выбираете сервер? Любая конфигурация на заказ!

Аренда физических серверов любых конфигураций под любые запросы - 1С бухгалтерия, игровые сервера, нагруженные проекты, интернет-магазины!

qwarta.ru подробнее

Size: a a a

Боль Тимлида

4460 membersпожаловаться на группу
2021 April 07

PD

Phil Delgyado in Боль Тимлида
code review отдельных, потенциально опасных, кусочков кода.
Например: sql-запросы длиннее пяти строк, использование примитивов многопоточности, доработка логики аутентификации/авторизации.
Review при этом делает эксперт (как роль) в конкретной задаче.
Список "опасностей" - заводится как чеклист и регулярно обновляется
источник
13:35пожаловаться#1

c

critskiy in Боль Тимлида
мыло и мочала начинай сначала
источник
13:35пожаловаться#2

Ms

Mutko says in Боль Тимлида
Эт способ борьбы со сложностью навязанных сверху процессов, тебе ж рассказывали про скрам снаружи, тимлидбот внутри)
источник
13:35пожаловаться#3

AD

Alexey Didukh in Боль Тимлида
Т.е. точечные нюансы, за которые переживает автор? А в других случаях никакие обсуждения-апрувы не нужны?
источник
13:36пожаловаться#4

VF

Victor Fabrichenko in Боль Тимлида
При этом, возможно проще и быстрее вообще сделать такие вещи в паре, ну или в дизайне заранее закрыть реализацию хорошим интерфейсом и потом внутрь специально обученный человек заглянет почекать.
источник
13:37пожаловаться#5

Ms

Mutko says in Боль Тимлида
В кровавеньком тырпрайзе преревью делается в личке, в ПР пишутся лишь позитивчек и лайки)
источник
13:37пожаловаться#6

PD

Phil Delgyado in Боль Тимлида
За которые переживает лид.
Если автора что-то волнует - то попросить ревью вообще правильный подход.
источник
13:37пожаловаться#7

Ms

Mutko says in Боль Тимлида
Карго культ код ревью)
источник
13:38пожаловаться#8

VF

Victor Fabrichenko in Боль Тимлида
Обсуждения нужны "до" и "по ходу", обсуждать уже сделанную работу, заранее знать, что сделаем и потом переделаем сразу с высокой вероятностью
источник
13:38пожаловаться#9

PD

Phil Delgyado in Боль Тимлида
Там по разному. Если есть сеньор и эксперт, то лучше попросить асинхронно проверить.
А если два миддла и никаких сеньоров не видно - то да, лучше синхронно в паре
источник
13:38пожаловаться#10

AD

Alexey Didukh in Боль Тимлида
А наличие потенциально проблемного кода определяется на дизайн ревью и ставится флажог ("не лить без кодревью?")
источник
13:38пожаловаться#11

Ms

Mutko says in Боль Тимлида
Зочем, пусть дербанит на мелкие куски и получает аппрувы на атомарном уровне
источник
13:39пожаловаться#12

AB

Alexander B in Боль Тимлида
лучше сразу в мастер мейн пушить анонимно!
источник
13:39пожаловаться#13

PD

Phil Delgyado in Боль Тимлида
Даже не на дизайн-ревью, а вообще чеклистом по команде. Типа "для кода, который делает такие-то вещи" - нужно специально попросить их проверить такого-то.
Не всю задачу, а отдельные куски.
источник
13:39пожаловаться#14

VF

Victor Fabrichenko in Боль Тимлида
Прочекать конкретные реализации тут и тут
источник
13:39пожаловаться#15

VF

Victor Fabrichenko in Боль Тимлида
К циклам претензий нет!
источник
13:40пожаловаться#16

PD

Phil Delgyado in Боль Тимлида
А, еще есть требования PCI DSS по ревью кода работы с карточными данными, там да, нужно ставить галочку на всю задачу и отдельно отписываться "все проверили".
Но это регуляторка
источник
13:40пожаловаться#17

AK

Anton Kucherov in Боль Тимлида
Интеграция кода и ревью - вещи ортогональные
источник
13:40пожаловаться#18

Ms

Mutko says in Боль Тимлида
Грамматика английского совершенна!
источник
13:40пожаловаться#19

AD

Alexey Didukh in Боль Тимлида
Круто! Вообще никогда не думал о таком подходе)
источник
13:40пожаловаться#20