Всех травить

[Бубновый Валет]

Я вообще человек добрый. Не люблю играть в политику. Но тут пришли некоторые перцы, со скиллами ниже среднего. Начинают откровенно в политику играть и самое главное заворачивать код ревью без особых причин. Ну как без особых - тикеты (ака сториз) у нас настоящее "хауноу". Тикет можно по-разному рассматривать. И если очень хорошо присмотреться, то можно всегда найти что там не сделано.

Ну так вот, некоторые начали присматриваться, требовать выполнения невыполнимого, менеджеру пох - сообщал об этом много раз ему, жаловаться выше пробовал - безрезультатно. Работу уже искать начал, но интересно как себя посоветуете вести.

Начал всех травить. Если раньше исходил из того, чтобы другим жизнь не отравлять, то сейчас придираюсь к каждой переменной, неправильно названному методу, вообще ко всему. Потому что могу показать зубы, и вообще не люблю когда обижают на пустом месте. Хорошо ли это или лучше одеть шубку белой овечки и позволить волкам себя терзать?

[adda_]

А за что вам код заворачивают?
То что вы написали об именах переменный и методах - чистый код вполне разумные требования, имхо

[Бубновый Валет]

Заворачивают за то, что не сделал то, что не написано в сториз. Т.е. история написана плохо, ну а додумать ее как бы твоя проблема)

[adda_]

Заворачивают за то, что не сделал то, что не написано в сториз. Т.е. история написана плохо, ну а додумать ее как бы твоя проблема)

Вы имеете ввиду что ТЗ не сформулировано корректно? Это общая проблема программирования.

Маркетолог спрашивает программиста: в чём сложность поддержки большого проекта?
Программист: ну представь, что ты писатель и поддерживаешь проект "Война и мир". У тебя ТЗ — написать главу как Наташа Ростова гуляла под дождём по парку. Ты пишешь "шёл дождь", сохраняешь, вылетает сообщение об ошибке "Наташа Ростова умерла, продолжение невозможно". Почему умерла? Начинаешь разбираться. Выясняется, что у Пьера Безухова скользкие туфли, он упал, его пистолет ударился о землю и выстрелил в столб, а пуля от столба срикошетила в Наташу. Что делать? Зарядить пистолет холостыми? Поменять туфли? Решили убрать столб. Получаем сообщение "Поручик Ржевский умер." Выясняется, что он в следующей главе облокачивается о столб, которого уже нет...

[valchkou]

чтобы избежать таких ситуаций в сториз во первых надо определиться с definition of done у вас в команде
во вторых для каждой стори прописывать acceptance criteria.
Добавления недопустимы (за очень редким исключением), должна создаваться новая стори
по поводу переменных должны же быть быть четкие стандарты в команде. шаг влево, шаг вправо -расстрел

[АццкоМото]

Очень странно, что полнота решения задачи — часть код ревью. Обычно не только не запрещается, но и поощряется коммитить неполный код. А полноту реализации проверяют по acceptance criteria куэйщики.

КМК нужно внедрять такую мысль начальству. Если не выйдет — прикинуться овечкой, искать работу, исподтишка саботировать

[АццкоМото]

чтобы избежать таких ситуаций в сториз во первых надо определиться с definition of done у вас в команде
во вторых для каждой стори прописывать acceptance criteria.
Добавления недопустимы (за очень редким исключением), должна создаваться новая стори
по поводу переменных должны же быть быть четкие стандарты в команде. шаг влево, шаг вправо -расстрел

Плюсую

[Мальчик-Одуванчик]

Если у пришедших перцев недостаточно навыков, то они сами подставятся.
Ваша задача их не спугнуть.

[_Lenchik]

Не на том уровне войну ведете.

Надо собирать доказательства их низкого профессионализма и раз в неделю ходить к менеджеру с намеком - ну совсем человек не тянет. Даже если менеджер отмахивается. Вода камень точит.

Я практически уверена, что они капают скопом на мозги менеджера против Валета. Скорее всего они посчитали, что Валет - это легкая мишень для политических игр и манипуляций.

[Мальчик-Одуванчик]

Я практически уверена, что они капают скопом на мозги менеджера против Валета. Скорее всего они посчитали, что Валет - это легкая мишень для политических игр и манипуляций.

Интересно какие вообще политические игры возможны в среде говнокодеров?

[Sergunka]

Заворачивают за то, что не сделал то, что не написано в сториз. Т.е. история написана плохо, ну а додумать ее как бы твоя проблема)

Индусы что ли?

[katit]

Я практически уверена, что они капают скопом на мозги менеджера против Валета. Скорее всего они посчитали, что Валет - это легкая мишень для политических игр и манипуляций.

Интересно какие вообще политические игры возможны в среде говнокодеров?

[major Major Major Major]

Очень странно, что полнота решения задачи — часть код ревью. Обычно не только не запрещается, но и поощряется коммитить неполный код. А полноту реализации проверяют по acceptance criteria куэйщики.

КМК нужно внедрять такую мысль начальству. Если не выйдет — прикинуться овечкой, искать работу, исподтишка саботировать

Кстати да. У нас то как коммит решает поставленную задачу это для ревью не слишком важно - на это есть QA которые и завернут ее обратно если не сделано. Бывают конечно исключения, но скорее из серии "а вот этот особый случай у тебя не разобран". Насколько чист код, соответствует стандартам, не изобретен велосипед, нет явных ошибок характерных для неопытных перцев. Могут возникать вопросы по масштабам изменений соответствующих задаче, к примеру чел чего то там по пути подчистил и подрефакторил - это не возбранятется, даже приветствуется но требует пояснений. Начальству можно сказать что попытка проверять соответствие решения и условий только отвлекают от проверки собственно кода.

[adda_]

Кстати да. У нас то как коммит решает поставленную задачу это для ревью не слишком важно - на это есть QA которые и завернут ее обратно если не сделано. Бывают конечно исключения, но скорее из серии "а вот этот особый случай у тебя не разобран". Насколько чист код, соответствует стандартам, не изобретен велосипед, нет явных ошибок характерных для неопытных перцев. Могут возникать вопросы по масштабам изменений соответствующих задаче, к примеру чел чего то там по пути подчистил и подрефакторил - это не возбранятется, даже приветствуется но требует пояснений. Начальству можно сказать что попытка проверять соответствие решения и условий только отвлекают от проверки собственно кода.

Рефакторить и подчисчать если не используется тест дривен дизайн, причем не фиктивно, а кейсами на всю бизнес логику крайне порочная практика. Очень легко можно поломать что то. У нас недавно это все официально было запрещено.

[ystar]

Очень странно, что полнота решения задачи — часть код ревью. Обычно не только не запрещается, но и поощряется коммитить неполный код. А полноту реализации проверяют по acceptance criteria куэйщики.

КМК нужно внедрять такую мысль начальству. Если не выйдет — прикинуться овечкой, искать работу, исподтишка саботировать

Кстати да. У нас то как коммит решает поставленную задачу это для ревью не слишком важно - на это есть QA которые и завернут ее обратно если не сделано. Бывают конечно исключения, но скорее из серии "а вот этот особый случай у тебя не разобран". Насколько чист код, соответствует стандартам, не изобретен велосипед, нет явных ошибок характерных для неопытных перцев. Могут возникать вопросы по масштабам изменений соответствующих задаче, к примеру чел чего то там по пути подчистил и подрефакторил - это не возбранятется, даже приветствуется но требует пояснений. Начальству можно сказать что попытка проверять соответствие решения и условий только отвлекают от проверки собственно кода.

Вот ваш коммит решает задачу как попало, а потом через полгода придется "столб" убирать.

Сколько раз уже замечено, если делать как попало, потом кастыль на кастыле и головная боль.

Вот у нас сейчас руководство решило все делать "правильно" и поставило это как самой главное задачей всем остальным сейчас дополнительной работы прилично будет, у меня на проекте, как работали так и будем без дополнительного головняка.

[Бубновый Валет]

Заворачивают за то, что не сделал то, что не написано в сториз. Т.е. история написана плохо, ну а додумать ее как бы твоя проблема)

Индусы что ли?

Нет, славные американские парни. Еще одна проблема в том, что у них сильно развит говорильный аппарат. Как что не так, включается говорилка, и своими оборотами, неологизмами и фразеологизмами они заваливают менеджера. Который, к слову, похоже скоро уйдет - есть инфа что его собираются уволить. Но думаю как бы он меня не уволил перед своим увольнением)

Короче, неприятные политические игры. Нет бы проблемы решать и кодить, большие дядьки, а занимаются ерундой.

[Sergunka]

Заворачивают за то, что не сделал то, что не написано в сториз. Т.е. история написана плохо, ну а додумать ее как бы твоя проблема)

Индусы что ли?

Нет, славные американские парни. Еще одна проблема в том, что у них сильно развит говорильный аппарат. Как что не так, включается говорилка, и своими оборотами, неологизмами и фразеологизмами они заваливают менеджера. Который, к слову, похоже скоро уйдет - есть инфа что его собираются уволить. Но думаю как бы он меня не уволил перед своим увольнением)

Короче, неприятные политические игры. Нет бы проблемы решать и кодить, большие дядьки, а занимаются ерундой.

Ну, а чо? Вас премии лишили или еще какая оказия подвернулась там вынесли строгий выговор с занесением в личное дело? На мой взгляд нормальная ситуация дайте людям самим себя в петлю завести. Просто документируйте все подробно в джире и будет Вам счастье когда наступит момент.

[Sanych]

Заворачивают за то, что не сделал то, что не написано в сториз. Т.е. история написана плохо, ну а додумать ее как бы твоя проблема)

Индусы что ли?

Нет, славные американские парни. Еще одна проблема в том, что у них сильно развит говорильный аппарат.

Тоже первая мысль была про индусов.
У нас на одном из проектов (софто-железный микс) нас четверо, и вот тоже такой есть славный американский с говорильным аппаратом. Через 5 секунд этой говорильни полностью теряется нить. Но есть одно но, оставшиеся трое - я, иранец и серб. И нам нам легко его затормозить методом типа "смотри в глаза и КРАТКО отвечай на вопросы". Плюс начальником, к счастью, именно серб...
Сочувствую
Какого возраста эти суперговорящие?

[АццкоМото]

Вот ваш коммит решает задачу как попало, а потом через полгода придется "столб" убирать.

Сколько раз уже замечено, если делать как попало, потом кастыль на кастыле и головная боль.

Вы подменяете теплое мягким. "Частичное решение" и "как попало" это вообще совершенно про разное.

Чтобы было понятно. Одна юзер стори - обычно несколько дней. А в один день - несколько коммитов. Совершенно очевидно, что коммит в типичном случае - маленькая часть стори.

Чтобы совсем понятно. Если юзер стори говорит "Нужно нарисовать красную квадратную кнопку, при нажатии на которую..." нормальный человек первым коммитом может сделать просто красную квадратную кнопку, которая не делает ничего. И всех умников, которые верещат, что задача не решена до конца, должен смело посылать в пешее. Если в команде это не так, большая беда с командой.

[Сабина]

Это пять !
А теперь представьте что вы работаете в конторе где ревью то никто не делает, да и вообще в гитхаб несильно чекинят. Вы один как идиот создаете себе PR, прописываете номера джир в комментах, мерджите Дженкинс себе прикрутили от ностальгии по нормальному девелопменту, maven repository ... и все это стоило нервов ....
И тут какой нибудь писатель говнокода или вообще безрукий вдруг встает на стендапе и говорит "а ну ка расскажи ка нам что ты там кодируешь". И ты как дурак готовишь презентацию, класс диаграммы, обьясняешь почему сделал именно так. И в ответ какой то невнятное безграмотное мычание.... А на следующем митинге - "делать будешь, как я сказал"

[Сабина]

Заворачивают за то, что не сделал то, что не написано в сториз. Т.е. история написана плохо, ну а додумать ее как бы твоя проблема)

Индусы что ли?

Нет, славные американские парни. Еще одна проблема в том, что у них сильно развит говорильный аппарат. Как что не так, включается говорилка, и своими оборотами, неологизмами и фразеологизмами они заваливают менеджера. Который, к слову, похоже скоро уйдет - есть инфа что его собираются уволить. Но думаю как бы он меня не уволил перед своим увольнением)

Короче, неприятные политические игры. Нет бы проблемы решать и кодить, большие дядьки, а занимаются ерундой.

Да уж, неприятно. Мне вот как то не попадались славные американские парни, помещанные на интригах. Интригами наоборот в основном интересовались борящиеся за место под солнцем недавние иммигранты. Из соотвествующих азиатских стран в смысле

[Сабина]

Через 5 секунд этой говорильни полностью теряется нить.

У него не Горбачев фамилия ?

[АццкоМото]

Через 5 секунд этой говорильни полностью теряется нить.

У него не Горбачев фамилия ?

Нить терял Черномырдин. Горбачев был просто витиеват, нор за нитью обычно следил

[Сабина]

Через 5 секунд этой говорильни полностью теряется нить.

У него не Горбачев фамилия ?

Нить терял Черномырдин. Горбачев был просто витиеват, нор за нитью обычно следил

То есть сам он следил ? Ну хоть так !
А то лично мне надоедало следить на первой половине его предложения ( не чтобы на тот момент было понятно только ли он начал предложение или уже почти закончил )

[Prosche]

Вот ваш коммит решает задачу как попало, а потом через полгода придется "столб" убирать.

Сколько раз уже замечено, если делать как попало, потом кастыль на кастыле и головная боль.

Вы подменяете теплое мягким. "Частичное решение" и "как попало" это вообще совершенно про разное.

Чтобы было понятно. Одна юзер стори - обычно несколько дней. А в один день - несколько коммитов. Совершенно очевидно, что коммит в типичном случае - маленькая часть стори.

Чтобы совсем понятно. Если юзер стори говорит "Нужно нарисовать красную квадратную кнопку, при нажатии на которую..." нормальный человек первым коммитом может сделать просто красную квадратную кнопку, которая не делает ничего. И всех умников, которые верещат, что задача не решена до конца, должен смело посылать в пешее. Если в команде это не так, большая беда с командой.

Абажите, но ревьювиццо ж не коммит, а говоря языком гита пул реквест или меж бранча в транк, если свн. Ктож коммиты то ревьювит?
Я так понимаю беда в том, что у ревьверов есть «видение» и оно не совпадает с авторским.