@prophp7

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

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

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
выбирать жертву и прописывать обязанности?

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
у человека завал, жопа горит, дедлайны просраны, а ему ещё ревью прилетает.
так вот он так и говорит что я занят до таковото времени ставь другова

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
для повышения качества, ожидаемо
кто им сказал что качество повысится? и кто дал такую гарантию

и на сколько оно повысится

как это измерить ?

что такое качество ?

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

которое ограничивает команду, хотя не во всех случаях

масштабировать человека сложно

Maksim
22.08.2018
11:21:51
кто им сказал что качество повысится? и кто дал такую гарантию
тот же, кто рассказал про вундервафлю под названием скрам\канбан и т.д. в любом производстве есть контроль качества. Он не гарантирует идеальной реализации, он старается минимизировать потери. Так же и у нас.

Admin
ERROR: S client not available

Dmitry
22.08.2018
11:22:33
которое ограничивает команду, хотя не во всех случаях
ну так команда не должна быть в 20 человек... команда это 5-7 человек

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
если в команде больше одного человека, лучше чтобы ревью было

Страница 1269 из 1387