Size: a a a

2018 December 12

MM

Maksim Maslov in JUG NN
Nick Kugaevsky
А как штатный разработчик заинтересован в качестве продукта, интересно узнать?
Я слышал что в этих ваших гуглах одна команда ведёт фичу на протяжении всего жизненного цикла
источник

SK

Sergey Kapralov in JUG NN
Roman Makhlin
Экспа должна быть в комманде
Почему не в проекте? (адекватный багтрекинг, ревью, ведение git-логов, техдокументации)
источник

SK

Sergey Kapralov in JUG NN
Sergey Kapralov
Почему не в проекте? (адекватный багтрекинг, ревью, ведение git-логов, техдокументации)
Буквально сегодня багу фиксил. Рут коз нашел, начал отслеживать откуда взялся рут коз. Нашел PR одной сотрудницы из штатов. Ни комментов, ни дескрипшена. Есть джира, а как фикс с джирой коррелирует непонятно. Сотрудница давно уволилась, ревьюверы нихрена уже по этому фиксу не помнят, и никто не может ответить на вопрос - почему ту давнишнюю багу пофиксили именно так а не иначе. А ответ нужен, потому что ткнешся переделывать - огребешь регрессий. Ну нахрен этот командный опыт.
источник

PM

Pavel 🦇 Malyshev in JUG NN
Sergey Kapralov
Буквально сегодня багу фиксил. Рут коз нашел, начал отслеживать откуда взялся рут коз. Нашел PR одной сотрудницы из штатов. Ни комментов, ни дескрипшена. Есть джира, а как фикс с джирой коррелирует непонятно. Сотрудница давно уволилась, ревьюверы нихрена уже по этому фиксу не помнят, и никто не может ответить на вопрос - почему ту давнишнюю багу пофиксили именно так а не иначе. А ответ нужен, потому что ткнешся переделывать - огребешь регрессий. Ну нахрен этот командный опыт.
Код ревью то не бывает что ли?
источник

SK

Sergey Kapralov in JUG NN
Pavel 🦇 Malyshev
Код ревью то не бывает что ли?
И че код ревью? Один ПР без дескрипшена я уже помянул. Ну закинул бы я свой фикс на код ревью. Толку от него если истории нет?
источник

SK

Sergey Kapralov in JUG NN
Хорошо еще кое как все таки догнал почему фикс был сделан так а не иначе.
источник

PM

Pavel 🦇 Malyshev in JUG NN
Sergey Kapralov
И че код ревью? Один ПР без дескрипшена я уже помянул. Ну закинул бы я свой фикс на код ревью. Толку от него если истории нет?
Дык на этапе код ревью и за PR без описания должны по рукам давать
источник

SK

Sergey Kapralov in JUG NN
Pavel 🦇 Malyshev
Дык на этапе код ревью и за PR без описания должны по рукам давать
Ну вот чето не дали. Меня в те времена в команде не было. Наверно тоже думали что опыт должен быть в команде
источник

PM

Pavel 🦇 Malyshev in JUG NN
Sergey Kapralov
Ну вот чето не дали. Меня в те времена в команде не было. Наверно тоже думали что опыт должен быть в команде
Раз не дали, значит код ревью нет. Без код ревью никакой передачи опыта не будет. И делай че хочешь
источник

SK

Sergey Kapralov in JUG NN
Pavel 🦇 Malyshev
Раз не дали, значит код ревью нет. Без код ревью никакой передачи опыта не будет. И делай че хочешь
Полностью согласен насчет пров без дескрипшена. Правда я все же склонен считать такие пры не средством обмена опытом между мемберами, а опытом внутри проекта.
источник

PM

Pavel 🦇 Malyshev in JUG NN
И так и сяк. Хорошая практика например, когда экспириенс-холдер, типа старожила проекта, проводит код-ревью и берет с собой кого-то из новичков.
источник

SK

Sergey Kapralov in JUG NN
Pavel 🦇 Malyshev
И так и сяк. Хорошая практика например, когда экспириенс-холдер, типа старожила проекта, проводит код-ревью и берет с собой кого-то из новичков.
Кто такой экспириенс холдер? И что будет с проектом если он намылится уйти?
источник

PM

Pavel 🦇 Malyshev in JUG NN
Sergey Kapralov
Кто такой экспириенс холдер? И что будет с проектом если он намылится уйти?
В том то и дело, если не делиться опытом, то будет капец. А если делиться, тогда можно и вырулить
источник

SK

Sergey Kapralov in JUG NN
Pavel 🦇 Malyshev
В том то и дело, если не делиться опытом, то будет капец. А если делиться, тогда можно и вырулить
Можно делиться опытом peer-to-peer, так сказать, но эффективнее складывать его в проект, линкуя между собой PRы, рут козы, солюшены, коммиты в гите, доки и митинг минутсы.
источник

SK

Sergey Kapralov in JUG NN
Чтоб по гит блейму можно было вычленить всю цепочку принятия решений
источник

SK

Sergey Kapralov in JUG NN
Это я и называю опытом внутри проекта
источник

SK

Sergey Kapralov in JUG NN
А опыт внутри команды - это когда "вот ту фичу Вася девелопил - только он в ней и может разобраться - иди спроси у него"
источник

PM

Pavel 🦇 Malyshev in JUG NN
В проект точно надо складывать, но общения отменять нельзя. А в твоем случае, девушка запиарила, ревью похоже не было и скорее всего никто кроме нее и не знал никогда почему именно так сделали.
источник

SK

Sergey Kapralov in JUG NN
Pavel 🦇 Malyshev
В проект точно надо складывать, но общения отменять нельзя. А в твоем случае, девушка запиарила, ревью похоже не было и скорее всего никто кроме нее и не знал никогда почему именно так сделали.
Мы спорим или говорим об одном и том же разными словами?
источник

RM

Roman Makhlin in JUG NN
Sergey Kapralov
А опыт внутри команды - это когда "вот ту фичу Вася девелопил - только он в ней и может разобраться - иди спроси у него"
Ну это в твоём понимании. Я вот в жизни не понимаю, чому чуваки из другого модуля или фичи должны мне о каждом шаге расшаркиваться - много шума.
источник