Разбор худшего пул-реквеста в моей карьере
За годы код-ревью я видел всякое, но один пул-реквест от мидла заставил меня проверить, не взломали ли наш гит. Задача была простой. Починить падение внешней интеграции в корзине. буквально 20 строк кода. Вместо этого автор принёс полторы тысячи строк изменений в десяти модулях. Он решил, что раз уж залез в код, то заодно перепишет всю логику оформления заказа, обновит старые библиотеки и внедрит модный паттерн, о котором только утром прочитал в интернете. Внутри этого хаоса творился настоящий ад. Вместо аккуратного перехвата ошибок там красовалась конструкция из пяти вложенных try-catch, которая просто глушила любые исключения, а в логи выводила лаконичное «Ой»...