Как мержить хотфикс в такой ситуации?

Question

[Stepan Yudin]

Всем привет!

Назрела ситуация:
В продакшене обнаружен баг. Создали ветвь хотфикса, и кое-где добавили, допустим, вызов метода SomeClass::someMethod(). Завершаем хотфикс и он мержится в мастер и девелоп ветки.

НО! Пока писали хотфикс, в девелоп ветку была смержена ветка фичи, в которой метод SomeClass::someMethod() был переименован в SomeClass::someMethodRenamed().

Что в итоге получается - в мастер ветке все ок, а в девелоп ветке будет вызов несуществующего метода (добавленный в хотфиксе).

Какой бест практис для таких кейсов? Благодарю.

Answer 1

[Толстый Лорри]

При мерже хотфикса будет конфликт, который будет разуливаться человеком, он сравнит коммиты и переименует.

А хорошая практика - чаще подмерживать общий develop в фичебранчи, чтобы такие штуки всплывали как можно скорее.

Comments

[jcmvbkbc]

> При мерже хотфикса будет конфликт, который будет разуливаться человеком, он сравнит коммиты и переименует.
Если "кое-где добавили, допустим, вызов метода SomeClass::someMethod()" то не будет никаких конфликтов.

[Толстый Лорри]

jcmvbkbc: ...и проблем не будет =) Речь шла о переименовании метода.

[jcmvbkbc]

> ...и проблем не будет
Будут проблемы во время сборки, поскольку в хотфиксе использовано старое название метода.

[Толстый Лорри]

jcmvbkbc: ну тогда компилятор спасет, если это не какой-нибудь мерзкий язычок с динамической типизацией.

[Stepan Yudin]

Кстати да, на стадии билда все схлопнется. При условии покрытия тестов.

[Толстый Лорри]

Stepan Yudin: Причем здесь тесты? Они только после билдов запустятся) Это статический анализ компилятора/линковщика сработает

[Stepan Yudin]

Толстый Лорри: это php тут не компилят)

[Толстый Лорри]

Stepan Yudin: Тогда о каком билде речь?