Code review discipline and working contracts — MPJ’s Musings Fun Fun Function

хорошее утро в понедельник сегодня я принимаю
некоторые вопросы с форума поклонников вентиляторов
художник программы, как управлять тягой
запросы в команде, как поддерживать тяну
запрашивает кучу как можно меньше, пока
сохраняя высокий фокус команды от одного
если вы хотите, чтобы это было с одной стороны
вы хотите, чтобы он был
как можно скорее с воздуха и на
с другой стороны, которую вы не хотите создавать
слишком много контекстных переключателей для вашей команды
участники да, получая хороший запрос на тягу
культура обзора кода в команде — это
это немного сложно установить
естественно, вы не должны тратить
немного работы, так что вы просто
просто будьте терпеливы, это требует
в то время как, но в последней команде я сказал
был включен в Spotify в настольной команде
у нас был гибкий тренер Джимми, который был
фантастический, и он был большим на эту вещь
называется рабочим контрактом и рабочим
контракт — это то, что вы разрабатываете
на собрании это обычно в форме
пост-это записка, и она говорит что-то
что команда соглашается сделать это не
задача — это скорее привычка, поэтому
пример в этом случае
Согласие было, я делаю кодовые обзоры каждый
день, который был рабочим договором и в
вещь с рабочим контрактом
что каждый человек в комнате
должны согласиться с тем, что это приоритетная задача
и что это то, что мы делаем или
вы не ставите этот рабочий контракт
он должен быть абсолютно единодушным, если
кто-то находится на заборе , в котором
убедить это лицо или если это лицо
не уверен вы положили , что работа
договориться и, как это, возможно,
колонке, где вы, где хорошо, мы собираемся
возможно, попробуйте, но не
каждый убежден , что это
хорошая идея, так что это очень много в пробной версии
режиме, и мы просто наблюдаем его за
Я не помню, что это было
в столбце hmm от верхней части моей головы
но в любом случае важная вещь
что он должен быть единодушным и как
неделя проходит, как через неделю или две
вы начинаете ставить
через рабочие контракты и
какие из них мы фактически делаем для
пример, который вы могли бы сделать, как мы всегда
написать единичные тесты для нашего кода могут быть
рабочий контракт, и вы просто наблюдаете
что вы это понимаете, и это не
действительно работают со старыми способами, которые мы выяснили
что хм у нас были, есть некоторые части
кода, который просто не подходит для
модульные тесты, или это просто безумный
причудливое количество работы, чтобы получить эту часть
тестируемого кода, поэтому мы имеем
подумал, что нет, мы не на самом деле
придерживаясь этого договора, поэтому мы решаем
либо , что мы дважды на него сверху вниз и
сосредоточьтесь на нем или мы решили сделать его более
реалистично, чтобы на самом деле
соответствует рабочему процессу, который
в этом случае мы решили изменить
это написать модульные тесты или написать очень
доброе оправдание, так что да, я всегда добавляю
модульные тесты или имеют очень хорошее оправдание
и это оказалось действительно хорошим
потому что тогда люди могут
вызов, который извините немного
Джастин, что вы избегаете случая, когда
вы просто не написать тест блока из
летаргии, но это также позволило нам
пропустить модульные тесты для случаев, когда
стоимость была слишком высокой, но
обратно в том случае , когда с
palookas, поэтому рабочий контракт там
что мы решили на
был обзор кода каждый день , так что это означает
что у вас реально не будет
подождать более одного дня для вашего кода
обзор должен быть выполнен часто
меньше, потому что каждый просматривает все
времени, и некоторые люди хотели бы
их обзорный блок утром некоторые
вы сделаете это только потому, что прежде чем они
Иди домой, так как они стали такими красивыми
красивое распределение и вы вы получили ваш
код рассмотрен в довольно целесообразном
Я думаю, что это оказалось
Довольно приятно, что это разрешено
каждый, чтобы создать рутину вокруг
потому что они могли выбрать
время суток, когда оно было подходящим для
их сделать обзор кода, как в
утром или днем, а также
позволил вам это вроде стать
эта вещь, где вы захлопнулись в своем
вещь , и где — то в течение дня и
то ты просто в порядке, там будет
как будто это займет день, прежде чем я получу
это так, теперь я иду делать что-то еще
поэтому просто отправьте их для этого
петля обратной связи , а затем я делаю что — то
иначе потребовалось бы некоторое переключение контекста
конечно, вот в чем суть
тянуть просит только другие реальные
альтернатива делать — то есть — это делать
пара-программирование, которое является чем-то, что
Я люблю, но в нем есть и другие
логистические проблемы и культура
проблемы это не так, это не так
тривиально, чтобы работать в команде так же, как
тянуть запросы или, но это здорово, если
Вы можете , если вы можете получить его , если вы хотите
взгляните на компанию, которая делает пару
хорошо программировать, вы можете посмотреть на основные
они делают это много, они даже не имеют
личные сеансы работы
общие рабочие станции, они только делают пару
программирование, которое круто, если вы попытаетесь
сделайте это быстрее, например ,
начать пинговать людей на слабину, чтобы сделать код
Отзывы вы начинаете в
рэп чужой работы, и это
это намного хуже, это создает
создает переключение контекста очень плохо
и я не думаю, что вам следует
это если у вас есть вытягивание
я думаю, что это было на
наименее лучший лучший поток, который у меня есть
видел рабочий контракт, что все
соглашается на то, чтобы я делал обзоры кода каждый день
вы просто смотрели эпизод веселья
веселой функции я выпускаю эти
В понедельник утром О 800 GMT вы можете смотреть
другой эпизод шоу, нажав
здесь, или если вы являетесь покровителем веселья
функции, вы можете обсудить это конкретное
вопрос на форуме Fun Fun
нажмите здесь Я ппд до следующего понедельника
утро остается любопытным