
Maksim
22.08.2018
11:03:12
Я не верю в ответственных людей. Просьбы не работают, либо часто сбоят. Нужны четки границы ответственности и тогда будет толк

Evgeniy
22.08.2018
11:03:41

Dmitry
22.08.2018
11:03:51
программисты пришли на работу программировать, а не разбирать чужой код. Так что это нормально, что им нет кайфа ревьювить

Maksim
22.08.2018
11:04:10

Google

Evgeniy
22.08.2018
11:04:21
все попытки в цифрах считать эффективность обычно проваливаются жестко

Maksim
22.08.2018
11:04:23
Я те мильон тасков в день в прод залью

Evgeniy
22.08.2018
11:04:54
есть области где kpi что то показывают но не везде

Ivan
22.08.2018
11:05:30
а если он сам себе тимлид?

Evgeniy
22.08.2018
11:05:33
само ревью, устроишься на работу к себе на удаленку с именем Антон например )))
а почему ревью идет только от тимлида ?

Maksim
22.08.2018
11:06:04
а как ты ревью пройдешь ?
Договорюсь с соседом, который будет код смотреть, что завтра он будет столь же эффективен.
Чувствуешь подвох?

Ivan
22.08.2018
11:06:14
чаще всего именно так. по скраму мало кто работает.

Evgeniy
22.08.2018
11:06:32
за задачу
я в разных компаниях видел пиздец какой разный скрам

Google

Dmitry
22.08.2018
11:07:14
ревью только на тимлида ... он мазохист...

Evgeniy
22.08.2018
11:07:14
мне кажется процессы работы зависят от проекта и команды
начинать надо с команды и проекта и тем что именно ревью может помогать команде
я бы выделил следующие пункты
1. О фитче будут знать несколько человек (помогает в случае с "bus factor")
2. Обучение людей которые делают ревью
3. Обучение человека кто сделал pr указанием на конкретные косяки ему.
4. Горизонтальное общение
имхо, у каждого свои пункты
кому то ревью не нужно

Dmitriy
22.08.2018
11:12:14
ну вы капец.. один говорит одну вещь, другой его понимает по-своему и с этим своим понимаем начинает спорить
в итоге такая каша )
я вот тоже не понял как грамотно организовать ревью

Maksim
22.08.2018
11:12:59
Разобрались же, всё живы.

Dmitry
22.08.2018
11:13:04
https://badootech.badoo.com/the-secret-to-great-code-reviews-4896639afc0e

Dmitriy
22.08.2018
11:13:15
выбирать жертву и прописывать обязанности?

Maksim
22.08.2018
11:13:42

Dmitry
22.08.2018
11:14:26
ну у нас пока автор выбирает ревьюверов, а конфликты или сам решает или на стендапе

Evgeniy
22.08.2018
11:14:47

Dmitriy
22.08.2018
11:14:59
конфликты мерджа или конфликты несогласия с его ревью?

Evgeniy
22.08.2018
11:15:04
прописывание обязанностей может привести к тому что люди перестанут говорить

Maksim
22.08.2018
11:15:17

Dmitry
22.08.2018
11:15:25
конфликты вида "не хочу/могу ревьювить"

Google

Dmitry
22.08.2018
11:15:41
конфликты несогласия с ревью решает течлид

Evgeniy
22.08.2018
11:15:52

Maksim
22.08.2018
11:16:24

Dmitry
22.08.2018
11:16:37

Dmitriy
22.08.2018
11:16:40
вово причем тут корона ваще не понял

Evgeniy
22.08.2018
11:16:47

Maksim
22.08.2018
11:16:56

Evgeniy
22.08.2018
11:17:05
ну тогда у вас лютый пиздец

Maksim
22.08.2018
11:17:07
в итоге автор кода отвлёк минимум 2х

Evgeniy
22.08.2018
11:17:17
либо им не интересно ревьюить тот код что ты написал

Dmitriy
22.08.2018
11:17:20
люди не любят конфликты, а ревью это повод для конфликта

Evgeniy
22.08.2018
11:17:54

Dmitry
22.08.2018
11:17:58
лютый пиздец у всех... если у вас не лютый пиздец, то или скучный бизнес или толпа беспризорных разрабов :)

Maksim
22.08.2018
11:18:11
имхо, ревью работает только в добровольно принудительном порядке... я никогда не видел что бы был толк иначе. Даже если в первое время энтузиазм по обмену работает, то всё равно перестаёт в 1 момент

Evgeniy
22.08.2018
11:18:51
зачем вводить в процес что то что мешает команде?
зачем для этого выдумывать способ чтобы они делали то что ИМ не нужно ?

Dmitry
22.08.2018
11:19:55
разработка идёт не ради команды, а ради бизнеса

Evgeniy
22.08.2018
11:19:59
и еще сверху надо review manager сделать который будет говорить типо за прошедшую неделю было созданно х ревью, прошло успешно у, столько то ждут продолжения
им ваще насрать на это

Google

Maksim
22.08.2018
11:20:23
для повышения качества, ожидаемо

Evgeniy
22.08.2018
11:20:40
и на сколько оно повысится
как это измерить ?
что такое качество ?

Dmitry
22.08.2018
11:21:12

Evgeniy
22.08.2018
11:21:29
которое ограничивает команду, хотя не во всех случаях
масштабировать человека сложно

Maksim
22.08.2018
11:21:51

Admin
ERROR: S client not available

Evgeniy
22.08.2018
11:22:22
тратят время разработки
нанимают людей

Maksim
22.08.2018
11:22:31

Dmitry
22.08.2018
11:22:33

Evgeniy
22.08.2018
11:22:35
покупают инструменты
а потом у разработчиков нет времени на основную работу
еще кое что я узнал, не делай хуйню, лучше ничего не делать, чем делать хуйню.

Google

Evgeniy
22.08.2018
11:24:36
а если этот кто то скажет вот в Х компании (или институте иследований и тд) пишут что если разработчиков подключить к электричеству и рандомно бить током то качество улучшиться и или совать морковку в определенное место
и они будут это внедрять, а главное ВЕРИТЬ в то что это работает ?
это не бизнес а религия уже получается имхо
ты назначаешь тимлида тимлидов
а еще лучше взять его со стороны

Dmitry
22.08.2018
11:26:50
руководитель разработки, да

Evgeniy
22.08.2018
11:26:56
на новую должность с максимальной зп

Maksim
22.08.2018
11:27:46
я чёт ход мысли потерял...

Dmitry
22.08.2018
11:27:53
тех дир назначает, когда сам не справляется

Evgeniy
22.08.2018
11:28:46
мое имхо довольно простое
если команде нужно ревью оно его внедряет и работает сама над ним, если команда по скилу не доросла до ревью, то внедрять его из под палки не лучшая идея.
ревью это отвественность разработки

Maksim
22.08.2018
11:29:10
команде ревью практически гарантированно не нужно

Evgeniy
22.08.2018
11:29:48
есть как бы труба с одной стороны приходят задачи с другой выходят
посредине разработка

Dmitry
22.08.2018
11:30:02
ну есть такие... которые в команду собрались типа работать "как крутые"... сами строят...

Evgeniy
22.08.2018
11:30:10
задачи - разработка - проверка - выкладка
если разработке нужно ревью оно его вводит
если не нужно оно не вводит

Dmitry
22.08.2018
11:30:50
обычно стартапы... 90% которых умрет до МВП, и еще 9% не выдержав конкуренции

Roman
22.08.2018
11:30:54
если в команде больше одного человека, лучше чтобы ревью было