От бага в чужом коде до смердженного PR

От бага в чужом коде до смердженного PR An architecture diagram generated by Archify. Нашёл проблему · баг или недостающий кейс · Моя работа · contributor Нашёл проблему баг или недостающий кейс contributor Изучил код, написал фикс · + регрессионный тест · Моя работа · contributor Изучил код, написал фикс + регрессионный тест contributor Открыл PR · с обоснованием решения · Моя работа · contributor Открыл PR с обоснованием решения contributor Ревью мейнтейнера · требования чужого проекта · Апстрим-проект · upstream Ревью мейнтейнера требования чужого проекта upstream Правки по замечаниям · цикл повторяется · Моя работа · contributor Правки по замечаниям цикл повторяется contributor Merge · в основную ветку проекта · Апстрим-проект · upstream Merge в основную ветку проекта upstream нужны правки повторное ревью approve Моя работа Апстрим-проект Legend Backend Database Security External

Чему это учит

  • • Читать чужой код и правильно локализовать причину, а не симптом
  • • Формулировать минимальный фикс, не трогая лишнего в чужом проекте
  • • Аргументировать решение под требования незнакомых ревьюеров

Почему это ценно на собесе

  • • Ровно то же самое, что проверяет секция код-ревью на техническом интервью
  • • Разница с учебным кодом: ревьюер видит проект впервые именно в этом PR