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