Shub
ну и пример стандарта, который определяет конкретные меры по нарушению dry было бы интересно почитать
Hog
праввила написания, принятые в данной компании
Shub
Hog
мы, кажется не про драй тут
Shub
все, что не форсится компилятором\фреймворком - это ни о чем
Shub
ну может быть линтеры еще
Hog
Shub
это может работать в маленьком коллективе высоко мотивированных людей
Shub
что дикая редкость
Hog
тебе только так кажется
Shub
Hog
нет - это про "ни о чём"
Shub
на практике особо тщательного ревьюера просто перестают добавлять в ревьюеры
Анна
Есть стопицот отмазок соскочить с принятых стандартов и послать нафиг ревьюера. Даже если ты мандатори ревьюер с правами цербера
Hog
это очень даже "о чём"
Shub
если команда больше чем на 5 человек, то там быстро формируются небольшие группки, которые будут аппрувить абсолютно все друг другу по принципу quid pro quo
Hog
Анна
при этом тебя тихо ненавидят все, чей код ревьюишь. И издеваются периодически
Hog
Hog
Shub
Shub
Анна
Если эти правила форсятся максимально автоматически и беспристрастно, то это упрощает жизнь всем участникам процесса, я считаю
Hog
если сформировались эти группки - надо разрешать социальный конфликт
Shub
Shub
ну и в целом, попытки решать проблемы бюрократии добавлением бюрократии обычно провальны
Hog
Вы правде в таком говне что ли работаете?
Shub
гриша, оглянись вокруг
Hog
Hog
Shub
я в широком смысле слова
Shub
ты работаешь с людьми. по статистике околок половины населения планеты тупее среднего
Shub
ты считаешь, у кодерков как-то иначе дело обстоит?
Hog
так код-стандарт и ревью тут только помогут
Shub
лол
Shub
как они помогут, если половина коллектива клала на них болт и не напрягаясь нашла 10 способов их обходить?
Shub
они везде есть. особенно там, где появляются карьеро-ориентированные персонажи
Hog
Анна
У нас к тому же ещё офигительное разнообразие с сотрудниками условно от 20 до 50+, Корея-Штаты-Беларусь, 35летние джуны и просто довольно много народу. Надо очень хорошо уметь работать с людьми, чтобы разруливать
Анна
Кодревью - одно из трудных мест в этом всём. Мы придумали некоторую схему наведения порядка, но даже так это непросто
Hog
Hog
Shub
а не становиться в пассивно-агрессивную позу типа "почему тут все мутабельно"
Shub
но для этого нужен тот самый домашний фреймворк\библиотека
Анна
Shub
но это лучше, чем лесоповал без фреймворка в принципе, не так ли?
Shub
да, разумеется
Shub
надо еще и консенсус в коллективе иметь, это вообще запредельно. щас же в дипломах пишут "самый умный в любой комнате"
Hog
Shub
интересная ты личность. с одной стороны говоришь про индустриальную разработку, а с другой стороны не знаешь, что у одного лида на проекте крайне ограниченные полномочия
Roman
Хороший срач, годный
Hog
полномочия твои зависят только от того, как ты их трактуешь
Shub
Hog
Roman
У нас есть ревью, но проблема в том, что пока следишь за мелочами, упускаешь общую картину новой фичи. Грубо говоря, проверили, что отступы правильные, тесты написаны, но фундаментальные архитектурные проблемы остались без внимания. Ибо сложно и много времени надо на качественное ревью, а ещё и код писать когда-нибудь нужно
Shub
воооо
Shub
сразу видно людей из больших коллективов
Roman
Shub
по уму ревью занимает примерно столько же, сколько занимает написать код
Hog
Shub
конкретно в моей команде 15 инженеров
Shub
это достаточный размер, чтобы я уже не знал, чем занят рандомно взятый коллега
Hog
Shub
куа к сожалению (и к счастью одновременно) всего 2
Hog
вот поэтому у тебя создаются группки, которые апрувят всё подряд
Shub
почему?
Hog
15 инженеров
Shub
как это связано?
Shub
типа, достаточно людей для создания подгрупп?
Hog
из 15 человек проще группки непересекающиеся создавать
Roman
Тут скорее не кол-во человек влияет, а уровень сложности входа в контекст пр
Shub
из 5 человек тоже можно сделать две непересекающиеся группы
Hog
Hog