Code review может быть большой болью для команды, которая только начинает его внедрять. Вы в любом случае наступите на много граблей: будете проводить ревью дольше, чем пишете код, устраивать смертельные споры про расположение скобочек и разбираться, можно ли сливать ветку в master до аппрува команды или нет. Я собрал ряд практик, которые помогут вам сделать процесс адаптации чуть менее болезненным — по крайней мере, мне они точно помогли.
Этот материал — краткая выжимка моего опыта, накопленного за несколько лет работы в крупных командах мобильной разработки. Опыт по большей части в мобильной разработке, что оказало влияние на используемые примеры и ссылки. Для тех, кто предпочитает не читать, а смотреть, в течение пары месяцев должно появиться видео с конференции Mobius, где я рассказываю доклад на эту же тему, но с кучей подробных практических примеров.
Читать полностью »
Рубрика «ревью кода» - 2
May the Code Review be with you
2017-06-14 в 11:58, admin, рубрики: code review, Блог компании Avito, код ревью, отладка, Программирование, разработка под iOS, ревью кода, Совершенный кодMust see: видеозаписи митапа MoscowJS 37
2017-05-31 в 12:38, admin, рубрики: 3d-моделирование, javascript, moscowjs, WebGL, webpack, анимация, Блог компании Avito, Программирование, Разработка веб-сайтов, ревью кода, метки: moscowjs
В четверг, 25 мая, в офисе Avito прошла очередная встреча сообщества фронтенд-разработчиков MoscowJS. Обсуждали отладку анимации, создание WebGL визуализации, сборку webpack’ом и код-ревью. Сегодня публикуем видеозаписи докладов — профессионалам будет интересно. Приятного просмотра!
Читать полностью »
Ревью кода в распределенной команде
2017-01-05 в 17:31, admin, рубрики: agile, Блог компании Mail.Ru Group, Карьера в IT-индустрии, никто не читает теги, Развитие стартапа, ревью кода, управление разработкой
Здесь описаны мои исследования, как сделать ревизию кода в команде более приятным занятием, которое может дать новый опыт всем участникам. У нас полностью географически распределённая команда, все коммуникации выполняются через интернет, и зачастую асинхронно. Мы используем Trello для описания возможностей продуктов, поодиночке создаём код, отправляем в GitHub пулл-реквесты, а также пользуемся встроенной в GitHub функцией их ревью. Это отличается от просмотра кода лицом к лицу в офисе и даже по видеочату.
Если не подходить к делу всерьёз, то асинхронная и письменная ревизия кода может стать причиной катастрофы в команде, приведя к ухудшению взаимодействия и сотрудничества. Но если все участники будут стараться делать всё хорошо, то такой подход может работать очень эффективно.
Читать полностью »
Уменьшаем количество ошибок с помощью чек-листа Code Review
2015-02-13 в 5:30, admin, рубрики: codereview, ит-инфраструктура, организация процессов, Программирование, разработка, ревью кода, метки: codereview, организация процессов, ревью кода В нашей статье об эффективных ревью кода мы рекомендовали использовать чеклист. Чеклисты (контрольные списки) — это великая вещь в ревью: они гарантируют, что ревью действительно прошло через вашу команду. Также они способствуют выявлению и решению общих трудностей.
Исследование, проведенное Software Engineering Institute, показывает, что программисты делают 15-20 распространенных ошибок. Добавив такие ошибки в чеклист, вы можете быть уверены, что заметите их в момент появления и поможете от них избавиться надолго.
Чтобы вам было от чего отталкиваться, вот вам список типичных пунктовЧитать полностью »
Эффективные ревью кода: 9 советов от исправившегося скептика
2015-02-13 в 5:20, admin, рубрики: codereview, ит-инфраструктура, организация процессов, Программирование, разработка, ревью кодаЯ знал теорию. Ревью кода помогает:
- Находить баги
- Обеспечивать читаемость и поддерживаемость кода
- Распространять знания кода на всю команду
- Позволять новым членам команды быстрее входить в работу
- Показывать всем новые подходы к решению задач
Или, это всего лишь пустая трата времени. По крайней мере, таким было мое первое впечатление от ревью кода.
Читать полностью »