SDB>>Image: approval.png
U>та ну... форматирование, сокращения в переменных, и т.п. дело 5 секунд...
майор очевидность сообщает что code review подвергаются коммиты менее 12 строк. бÓльшие коммиты вызывают переполнение буфера и аппрувятся без code review.
Здравствуйте, Stanislaw K, Вы писали:
SK>майор очевидность сообщает что code review подвергаются коммиты менее 12 строк. бÓльшие коммиты вызывают переполнение буфера и аппрувятся без code review.
Везет тем у кого так. Недавно на одном ПРе собрал 132 комента (при этом 2 комента были по делу).
Есть логика намерений и логика обстоятельств, последняя всегда сильнее.
Здравствуйте, VladD2, Вы писали:
SK>>майор очевидность сообщает что code review подвергаются коммиты менее 12 строк. бÓльшие коммиты вызывают переполнение буфера и аппрувятся без code review.
VD>Везет тем у кого так. Недавно на одном ПРе собрал 132 комента (при этом 2 комента были по делу).
Остальные 130 что, (вместо того, чтобы работу работать) обсуждали какую пиццу заказать ?
Здравствуйте, VladD2, Вы писали:
SK>>майор очевидность сообщает что code review подвергаются коммиты менее 12 строк. бÓльшие коммиты вызывают переполнение буфера и аппрувятся без code review. VD>Везет тем у кого так. Недавно на одном ПРе собрал 132 комента (при этом 2 комента были по делу).
Такая же ерунда иногда бывает, только на пару десятков 1-2 по делу.
SK>>Остальные 130 что, (вместо того, чтобы работу работать) обсуждали какую пиццу заказать ?
U>ну например комменты в стандартных шаблонах типа "тут вставляем бла-бла-бла. заменяем блаблабла на блаблабла"... например.
Это не 130 строк твоих комментов, это 130 постов-комментов тех, кто делает review твоего коммита.
Здравствуйте, undo75, Вы писали:
U>ну например комменты в стандартных шаблонах типа "тут вставляем бла-бла-бла. заменяем блаблабла на блаблабла"... например.
Опыт показывает, что на простую тему все хотят высказаться, показать свою эрудицию. А вот если задать сложный вопрос, желающих ответить будет найти нелегко.
Большой коммит на 500 строк, с тестами, документацией, объяснением почему и как. В правильном стиле, легко поддерживаемый. От грамотного инженера, которому уже доверяешь. Зачастую третья версия (первые две были драфты, по которым понятно, что человек думал над написанным).
И 10 строк "а сейчас мы сделаем дыру через все слои, и намертво приклеим интерфейсы к реализации". Потому что "я художник, я так вижу". Разумеется, без тестов и документации, в 10 строк не поместится. Дальше 100500 комментов на тему "так делать нельзя" и столько же ответов "мне очень нужно, я хочу чтоб уже завтра в релиз, потому что performance review cycle начинается послезавтра".
Pzz>Опыт показывает, что на простую тему все хотят высказаться, показать свою эрудицию. А вот если задать сложный вопрос, желающих ответить будет найти нелегко.
Практика показывает, что "сложный вопрос" обычно является X-Y проблемой, и требуется нетривиальное количество времени, чтобы вопрошающий это осознал.
Также бывает, что "сложный вопрос" требует многошагового анализа "как же вопрошающий дошел до этого вопроса", или, корпоративным языком, "контекст". В большинстве случаев оказывается, что где-то было принято очередное решение из серии "здесь рыбу заворачиваем, потому что лень разбираться", и вопрос на самом деле не имеет смысла.
...но бывают и в самом деле непростые или неочевидные вопросы. Не те, где "у нас 100 сущностей и каждая может взаимодействовать с каждой, поэтому никто не знает, что происходит в системе". А те, где сущностей обозримое количество, и взаимодействия ожидаемые. Но что-то идет не так...
