Сучасна диджитал-освіта для дітей — безоплатне заняття в GoITeens ×
Mazda CX 30
×

Що у вас в командах роблять з overperformers?

Ситуація, є команда працює в певному темпі, стараються дивитися на якість свого коду тестити, і є людина дуже спішить і любить рефакторити. І може воно би непогано, але часто виходить код який потім ламає усе. Але людина вище рангом і код рев’ю зазвичай закінчуються довгими спорами або продавлюванням своєї думки. В результаті команда в ’’пі, а він молодець бо свої таски деліверить швидко. Що робити?

👍ПодобаєтьсяСподобалось1
До обраногоВ обраному1
LinkedIn
Дозволені теги: blockquote, a, pre, code, ul, ol, li, b, i, del.
Ctrl + Enter
Дозволені теги: blockquote, a, pre, code, ul, ol, li, b, i, del.
Ctrl + Enter

Стикався з подібним але тіп ще пи*дів менеджерам що то його підлеглі не шарять, навіть пару раз приходилось збирати банду інженерів зі сторони щоб добитись нормальної реалізації задачі :)

Таких гарних спеціалістів треба віддати комусь ще або звільнити

людина вище рангом

Знакомо такое, да, от таких мудаков только надо очищать команду либо опускать ниже рангом, т.к. он ломает нормальный воркфлоу. Лучше избавиться от одного, чем потерять всю или часть команды, т.к. с такими очень трудно работать, а еще он в случае факапа на проде может все свалить на других разрабов, а факап или залет у вас по-любому случится с таким подходом. Так вот этот хрен мерджил все в мастер без спросу, так у него были права, а поломанные юнит тесты просто комментировал, они потом так и валялись в репе месяцами закомментированными.
Тесты не помогут, тут проблема в корне, что продавливается свое мнение, в стиле «вот белый лист, но я сказал что он черный, значит черный и не обсуждается». Хороший лид прислушивается к конструктивному обсуждению своего кода

Уволить по хорошему надо

тут вже писали, що якщо хтось все ламає то проблема в тому, що у вас базових тестів нема. тобто потрібні тести.
але більш практично це ввести правило що всі зміни/коміти мають покривати тестами те, що змінюється. тобто якщо цей чувак херачить купу всього, рефакторінг то він МУСИТЬ покрити все це тестами, пул реквести без тестів не апрувляться!
це все дуже базові, дефолтні речі в розробці, якщо в вашій конторі так не заведено, то це гівно-кантора. якщо людина — лід і досі таке не впровадила, то це гівно, а не лід.

В мене пару питань:

1) є у вас якийсь CI/CD, шоб оте «ламає все» злапати зразу й надавати пендюлів? а чо нема? Без цього весь ваш пост сприймаєцця як бездоказове «ну а шо він тут»

2) зробіть аналіз, чи чувак не робе так багато тасків, бо спочатку наробе собі проблєм, а потім героїчно їх порозвʼязує, тикніть колєґу носом і скажіть, шо, по-перше, slow is smooth and smooth is fast, а по-друге, в будь-якій ситуації, коли маєш дві зміни в коді, які розвʼязують одну й ту саму проблему з однаковою якістю, виграє та, з якої пач меньший

3) я видів ситуації, коли хтось дійсно довозив багато тасків, в тому числі й з рефакторуванням, але інші колєґи не мали доброї звички робити на початку дня аналога `git pull origin main` (підставте своє, ребейс чи шо там у вас прийнято), а потім за день до релізу своїх речей скіглілі, шо в них конфлікти повилазили, і чо той там поперед батька в пекло спішив. Ну а їхні гілочки старі по пару місяців, не завжди же ж всьо довезеш за один спринт. У вас часом не таке? Деякі ше робили розвʼязування конфліктів таким способом, шо вже довезені фічі або поправки тихачем пропадали з коду.

загазлайтити, щоб він думав, що він лоуперформер)

Накльопаний код не повинен проходити код рев‘ю і все.

В результаті команда в ’’пі, а він молодець бо свої таски деліверить швидко. Що робити?

Якщо ти просто дев і інші деви думають ± то саме то пожалітись ліду.
Якщо ти лід і ти так думаєш або тобі жалілись на нього то перший раз поговорити 1 на 1, чітко і прямо, типу ти багато робиш але й багато ламаєш. Роби акуратно і не спіши.
Якщо не допомогло, то давати складні таски або (якщо таких нема) то чмирити в код рев’ю всією тімкою.
Дойо##тись до кожної коми і пробілу. І на кожен другий рядок питатись «а чому так в не через рекурсію/сортування бульбашкою, чого сам написав а не лібку заюзав і навпаки».

Навіть до тих що в танку дойде що тімці така поведінка не подобається і людина або виправиться або свалить сама.

Кароч, то саме що і з лоу перформерами, гнати того задрота з тімки щоб потім всім не прйшлось більше 4 годин в день працювати.

перші поради мають рацію, а закінчив мудацьким дном

Десь тут повинен пробігти Кожаєв з «Ничего не делают, у меня нож есть» :D

Скрам-майстер приносить його в жертву біля канбан-дошки, після чого кожен з’їдає трохи серця, мізків та печінки, щоб перформанс перейшов до нього та наповнив силою. Після того на тімбілдінгу смажать шашлики, черепом грають в кеглі після роботи.

а кальянну шахту з якої частини тіла роблять?

після чого кожен з’їдає трохи серця, мізків та печінки

Кажуть, за поїдання мізків пріоновий боженько покарає

Чотири рядки не достатньо для єдино правильного діагнозу: описана людина ні в якому разі не оверперформер. Може, дорвався до маленької влади ліда і кайфує, а може він єдиний у вас в команді кого якість коду хоч троха турбує :) Спробуйте дати йому почитати розумні книжки про рефакторінг, скажімо; по книжках, код після рефакторінгу не повинен ламатись :)

часто виходить код який потім ламає усе

Якщо він «вище рангом», дійсно не правий і не розуміє аргументів, то зрештою підставте його пару раз перед замовником :(( шо простіше...

Це не оверперформер, а ковбой-гівнокодер. Лікується дефінішн оф дан на тасках. Бажано, заенфоршеного бездушною машиною. Для початку рекомендую 70% ковереджу на новий код, цикломатична складність менше 12, без нових код смеллів, ворнінг == еррор. Якщо людина давить в коментах на код-ревью, запрошуєте незацікавлене авторіті, резолвите більшістю голосів.
Баги тріажите регулярно, вішаєте на того, хто його зробив. Пару спринтів порозгрібає те, що зробив — буде обернене тому, що ви описали

команда в ’’пі, а він молодець

. Тому що нових фіч запедалив 0, а баги фіксав, що сам зробив.
Не спрацює в кейсі, коли менеджер тіми — його кум, там навіть якщо він буде навалювати посеред кімнати —

він молодець бо свої таски деліверить швидко

Звиняйте, хлопці, англомовна команда, тому досить рідко використовую аналоги з мови ( та й перейшов недавно з тої-що-не-можна-називати)
дефінішн оф дан — визначення зробленого
занфоршеного — примусово визначеного
ковередж — покриття
код смелл — вонячий шматок коду
авторіті — ваш кум
тріаж — прошу допомоги
фіча — штука
менеджер тіми — атаман куреню.

тріаж — прошу допомоги

я думав це сортування, типу за критичністю, по аналогії з медичним тріажем)

Я в такій же ситуації з мовами, як і ви, повністью зрозумів про що ваш меседж, і навіть не допер одразу про що був меседж з паундом сосіджей, лише коли ви вже почали відповідати )))). Особливо в голові каша коли три мови отак мішаються то дзівнок, то робота, то дім і потім вже таку маячню несешь шо капец :-)

А тести?
Нагавнякати полотно коду це нехитра справа.
А перевірити це? А покрити тестами? Введіть правило, що таска, код якої непокритий тестами не вважається виконаною.
Нехай рефакторить, це корисно, але все що нарефакторив має покриватись тестами.
Ну і правило: «сам поламав, сам фіксай» ще ніхто не відміняв.

Хай перепише хоч все, але несе за це відповідальність і покриває все тестами.

Але людина вище рангом

Або заб’є, або перекине на когось, а він молодець.

Треба менеджера долучати.

треба вийти з такими питаннями в інтернет, аімєнно на двач. Я чув там можуть щось порадити.

а він молодець бо свої таски деліверить швидко.

як казав пан арлазоров «вам посчитать быстро, или правильно?»

Це не оверперформер. Оверперформер — це класичний 10x programmer.

А вирішується тим, що людина закріплена за модулем, і в чужий код не лазить.

І ще в гіті придумали бранчі Може рефакторити там.

Дайте більше інформацію про вашу ситуацію, може пораджу щось краще.

Розслабся я думаю що це тролінг сабжа про ловперформерів.

Та я й не напрягаюсь.

А розслабляюсь по повній в паралельній гілці.

Гноблять А взагалі то людина ще на спрацювалась із командою банально. Рефкторити то не погано, погано коли воно при цьому усе зламалось і довелось доволі довго лагодити, якщо швидко полагодив — то і проблема нема.

1. Правильне планування. Щоб паралельно не було тасків, код яких сильно перетинаеться. Це сильно спростить мержі.
2. Планувати не великі за об’ємом та часу на розробку таски/юзер сторі
3. Домовитися, що рефакторити тілько код, який відноситься безпосередньо до поточної задачі
4. Робити окремі таски/юзер сторі для рефакторінгу чогось, що вирішила на мітингах команда
5. Підмержувати мастер/мейн до фича бранчу, а не чекати пул ріквесту
6. Розмовляти, щоб налагодити процеси, які будуть зручні саме для вашої команди
7. Якщо ж вже вирішили брати таски в паралель, що 100% будуть мати значну частину спільного коду — то змиритися, що хто перший замержив — той і не буде мати проблем з мержем

А хто фіксить код поломаний?

Віддають лоу перформерів в рабство
Кадровиця нажаль дістаються менеджеру

Що робити?

Без хірургічного втручання по вилученню шила з задниці тут ніяк

Підписатись на коментарі