Какие антипаттерны или типичные ошибки ты замечал при проверке merge request'ов? На что обращал внимание?

Назовите реальные типичные проблемы: неясные проверки, дубли, скрытые зависимости, нестабильные ожидания и небезопасные данные. Объясните, как замечание помогало улучшить результат, а не только стиль.

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