Запрошенная повторная публикация из StackOverflow:
Я работаю в небольшое время разработки с очень ограниченным временем для разработки. Мы разрабатываем инструмент, который важен для результата нашей работы, но не используется ежедневно. Я единственный человек в команде, имеющий опыт работы программистом.
Моя проблема в том, что я настаивал на обзорах кода перед слиянием с транком более года. Все согласились с этим, но все же только мой код был рассмотрен. Вернувшись из долгого отпуска, я возвращаюсь к сундуку с комментариями кода, так как «это уродливое решение - удалите как можно скорее» и «быстрое исправление». Новым является то, что за инструмент назначен парень. (Роль, которая была впервые предложена мне, но я отказался из-за не связанной с работой причины.) И он считает, что это нормальный способ работать: поскольку у нас так мало времени, чтобы развиваться, мы должны так срезать углы.
Меня беспокоит то, что другие разработчики пишут некрасивый код: часто нарушают инкапсуляцию, пишут огромные классы, добавляют внутренние классы в незнакомых местах, проводят мало или вообще не проводят модульные тесты и так далее. В конечном итоге будет невозможно развивать инструмент дальше.
Должен ли я настаивать на том, чтобы мы выполняли проверку кода перед слиянием с транком или я просто сука качества кода?