Открыть список
Как стать автором
Обновить
7
Карма
0
Рейтинг
Иван Садовой @greblin

Пользователь

Code review по-человечески (часть 1)

И еще вдогонку
Хотелось бы еще поинтересоваться опытом коллег по части немедленного проведения CR. С одной стороны всё правильно — блокируется работа коллеги, задача зависает перед отправкой на прод и т.д. Но с другой стороны переключение контекста может быть весьма дорогим и неприятным для разработчика, да и какая-нибудь рабочая встреча в самый неподходящий момент случится.

У себя в команде (8 разработчиков) мы ввели несколько правил:
  1. ревьювить может любой разработчик, и аппрувы всех равны, независимо от опыта и длительности пребывания в команде;
  2. для мерджа в мастер необходимо минимум два аппрува;
  3. ни один комментарий не должен остаться без ответа: коммита с исправлением или достигнутого вместе решения что можно не править

С одной стороны это хорошо: здоровая атмосфера и восприятие критики в команде, обмен опытом, в котором может участвовать каждый, чаще обнаруживаются баги и улучшения, CR проходят за приемлемое время. Но с другой стороны размывается ответственность, люди чаще думают «А ладно, посмотрю попозже, или еще кто-нибудь посмотрит», или ревьювят сквозь пальцы. И вот уже немедленной реакции нет, одну задаче прямо хорошо поревьювили, а другую «пойдёт» и т.д.

Кто как находит баланс в этой проблеме?

Code review по-человечески (часть 1)

Спасибо за отличную статью! Разрешите предложить ещё одну крутую вещь, которую можно включить во вторую часть, хотя она больше относится к прохождению CR, а не проведению: раннее code-review. Например, спроектировали интерфейс — не надо сразу бросаться его реализовывать, покажите на code-review.

44 урока управления технарями

Полностью согласен с Вами. Для тимлида команды на 5-7 человек я бы 7 пункт переформулировал так:

Помните, что теперь у вас появилось много не менее важных задач, и писать код (тестировать, проводить аналитику, etc) теперь вы будете меньше. Не пытайтесь всё сделать в одиночку, делегируйте задачи. И ни в коем случае не забирайте себе все самые интересные задачи

Обратная сторона авиабилета. Как Туту.ру помогает подобрать оптимальный тариф

Да, вы его не видите, потому что из всех тарифов с одинаковыми опциями мы показываем только самый дешевый. И в данном случае получилось так, что тариф BOW2 с ограничениями в 10кг на багаж дороже тарифа NOW (это за 7430), у которого разрешенный багаж — 20кг

Обратная сторона авиабилета. Как Туту.ру помогает подобрать оптимальный тариф

M2 в данном случае получается дороже из-за того, что с тарифом Ямала не может быть скомбинирован самый дешевый тариф Нордстара — на М2 комбинация тарифов BFHOW и BOW2, а по отдельности — BFHOW и NOW2. А если взять сумму BFHOW и BOW2, то суммы совпадут.

По второму вопросу — да, в ряде случаев имеет смысл поискать сегменты по отдельности. Особенно это может быть заметно на комбинациях внутреннего и заграничного рейсов. Но надо помнить, что при такой покупке авиакомпании не несут взаимной ответственности: авиакомпания второго сегмента не пересадит пассажира на другой рейс, если рейс на первом сегменте опоздал и стыковка нарушилась. Также придется получать багаж и заново регистрировать его. В некоторых случаях могут возникнуть еще и визовые трудности

Обратная сторона авиабилета. Как Туту.ру помогает подобрать оптимальный тариф

Да, мы продаем М2. Вот, например, здесь первое предложение от Ямала и Нордстара — М2

Обратная сторона авиабилета. Как Туту.ру помогает подобрать оптимальный тариф

Такая идея есть. Но и в GDS сейчас развиваются инструменты, которые позволят авиакомпаниям заносить максимум информации в едином формате

Информация

В рейтинге
5,833-й
Откуда
Москва, Москва и Московская обл., Россия
Работает в
Дата рождения
Зарегистрирован
Активность