Size: a a a

2021 March 02

СА

Сергей Аксёнов... in ctodailychat
Всем привет! Вот ещё вопрос на обсуждение. Знаете ли вы команды, где процесс code review имеет более высокий (или равный) приоритет, чем выполнение собственных задач? Я слышал, что якобы в Google именно так. Может вы в своих командах такое внедряли, но не получили нужного эффекта?
источник

A

Artur in ctodailychat
Сергей Аксёнов
Всем привет! Вот ещё вопрос на обсуждение. Знаете ли вы команды, где процесс code review имеет более высокий (или равный) приоритет, чем выполнение собственных задач? Я слышал, что якобы в Google именно так. Может вы в своих командах такое внедряли, но не получили нужного эффекта?
внедряли в разных команда. конечно, в рамках разумного, т.е. приоритет был "не бросай все немедленно и делай ревью", но что-то вроде сделать ревью перед тем, как уйти домой, или первым делом с утра, пержде, чем браться за свою задачу. эффект - как от ежедневной физкультуры: конечный результат всем нравится, но каждый раз хочется покапризничать и съехать)
источник

O

Oleg in ctodailychat
Сергей Аксёнов
Всем привет! Вот ещё вопрос на обсуждение. Знаете ли вы команды, где процесс code review имеет более высокий (или равный) приоритет, чем выполнение собственных задач? Я слышал, что якобы в Google именно так. Может вы в своих командах такое внедряли, но не получили нужного эффекта?
У нас по сути так, ну т.е. от команды к команде по разному. Кто-то выделяет прям время на ревью сразу после стендапа, т.е. все что накопилось за вчера. Кто-то ( у кого канбан) прям лимиты ставит и команда не берет новую задачу пока есть задачи в ревью. Так или иначе чем дальше задача продвинулась по процессу, тем более приоритетна она
источник

Y

Yaroslav in ctodailychat
Artur
внедряли в разных команда. конечно, в рамках разумного, т.е. приоритет был "не бросай все немедленно и делай ревью", но что-то вроде сделать ревью перед тем, как уйти домой, или первым делом с утра, пержде, чем браться за свою задачу. эффект - как от ежедневной физкультуры: конечный результат всем нравится, но каждый раз хочется покапризничать и съехать)
Еще можно уйти дальше и сделать пост мердж код ревью. Утром смотрим дифф мастера
источник

Y

Yaroslav in ctodailychat
Но это при условии что код ревью замещается дизайн ревью
источник

SS

Slava Savitskiy in ctodailychat
Сергей Аксёнов
Всем привет! Вот ещё вопрос на обсуждение. Знаете ли вы команды, где процесс code review имеет более высокий (или равный) приоритет, чем выполнение собственных задач? Я слышал, что якобы в Google именно так. Может вы в своих командах такое внедряли, но не получили нужного эффекта?
вот мы сегодня это обсуждали, все заняты работой и реквесты висят по неделе. договорились выделять время после стендапа каждый день и проверять, нет ли нового кода на ревью
источник

SS

Slava Savitskiy in ctodailychat
Yaroslav
Еще можно уйти дальше и сделать пост мердж код ревью. Утром смотрим дифф мастера
😱 это если у вас в компании 5 разработчиков работает?
источник

Y

Yaroslav in ctodailychat
Slava Savitskiy
вот мы сегодня это обсуждали, все заняты работой и реквесты висят по неделе. договорились выделять время после стендапа каждый день и проверять, нет ли нового кода на ревью
Есть еще програмное рещение от гитпрайм, они метрики трекают ;)
источник

Y

Yaroslav in ctodailychat
Slava Savitskiy
😱 это если у вас в компании 5 разработчиков работает?
Да
источник

f🤔

focusshifter 🤔 in ctodailychat
Сергей Аксёнов
Всем привет! Вот ещё вопрос на обсуждение. Знаете ли вы команды, где процесс code review имеет более высокий (или равный) приоритет, чем выполнение собственных задач? Я слышал, что якобы в Google именно так. Может вы в своих командах такое внедряли, но не получили нужного эффекта?
у нас были правила “вечером не должно быть висящих ревью, если они есть - то задача их провести становится первоочередной и блокирует остальные на следующий день”, работало
источник

АА

Александр Арбузов... in ctodailychat
Artur
внедряли в разных команда. конечно, в рамках разумного, т.е. приоритет был "не бросай все немедленно и делай ревью", но что-то вроде сделать ревью перед тем, как уйти домой, или первым делом с утра, пержде, чем браться за свою задачу. эффект - как от ежедневной физкультуры: конечный результат всем нравится, но каждый раз хочется покапризничать и съехать)
полностью согласен, что они должны быть разумно равны как свои задачи.
у нас была ещё возможность поднять приоритет ревью меншеном на стендапе/в чатике. с мотивацией, чтобы по возможности блокировать меньше других девов и qa.
источник

O

Oleg in ctodailychat
Oleg
У нас по сути так, ну т.е. от команды к команде по разному. Кто-то выделяет прям время на ревью сразу после стендапа, т.е. все что накопилось за вчера. Кто-то ( у кого канбан) прям лимиты ставит и команда не берет новую задачу пока есть задачи в ревью. Так или иначе чем дальше задача продвинулась по процессу, тем более приоритетна она
некоторые команды еще бота юзают, которые смотрит открытые MR и в зависимости от давности пишет в чат команды всякие сообщения (он лежит на github, не знаю можно ли сюда ссылки кидать, чтобы не сочлось за рекламу)
источник

Y

Yaroslav in ctodailychat
Oleg
некоторые команды еще бота юзают, которые смотрит открытые MR и в зависимости от давности пишет в чат команды всякие сообщения (он лежит на github, не знаю можно ли сюда ссылки кидать, чтобы не сочлось за рекламу)
Нужно
источник

O

Oleg in ctodailychat
Yaroslav
Нужно
источник

K

KivApple in ctodailychat
Toвapищ Maйop
Есть ли какие-нибудь юридические способы решения. Если кто сталкивался.
Боюсь технические средства тут будут надёжнее
источник

ES

Egor Suvorov in ctodailychat
Сергей Аксёнов
Всем привет! Вот ещё вопрос на обсуждение. Знаете ли вы команды, где процесс code review имеет более высокий (или равный) приоритет, чем выполнение собственных задач? Я слышал, что якобы в Google именно так. Может вы в своих командах такое внедряли, но не получили нужного эффекта?
В Google на двух стажировках я наблюдал скорее равный приоритет. То есть можно задачи поделать, а можно сделать code review, при этом всё это нужно обязательно. Code review, может, даже, обязательнее, потому что коллег блокирует.
источник

ES

Egor Suvorov in ctodailychat
Но его делать не так прикольно, да и CL (changelist) бывают большие, которые хорошо бы сначала голосом обсудить, поэтому всё-таки не мгновенная реакция.
источник

И

Илья in ctodailychat
Есть задачка, если лист 200.000 url, нужно все проверить по коду 404 или 200. И получить файл с 200.
Есть что-нить готовое? Как проще решить?
источник

Y

Yaroslav in ctodailychat
Илья
Есть задачка, если лист 200.000 url, нужно все проверить по коду 404 или 200. И получить файл с 200.
Есть что-нить готовое? Как проще решить?
Разово или мониторить?
источник

И

Илья in ctodailychat
Yaroslav
Разово или мониторить?
Разово
источник