Наш менеджер отслеживает коммиты Git на всех наших проектах; обычно это не проблема, и мне нравится тот факт, что контроль версий предоставляет журнал всей происходящей работы, особенно для последующего аудита и анализа (на случай, если что-то пойдет не так).
Однако менеджер сделал несколько комментариев, спрашивая, над чем работают люди, когда он видит фиксацию, которая читает «исправления стиля», или любое сообщение фиксации, которое не ссылается на номер заявки в нашей системе управления задачами.
Есть социальное или техническое решение для этого?
Дополнительная информация: это проект технического обслуживания, так что есть куча «пришлось сделать A, затем B, затем C, а затем, наконец, приступить к реализации X».
Больше информации: конкретное сообщение о коммите, которое подняло флаг с менеджером, было близко к «включенному лучшему способу X, Y и Z», которое является скорее сообщением рефакторинга, а не простым стилистическим исправлением.
источник
Ответы:
Полагаю, но это не проблема .
Ваш менеджер должен знать, что вы, ребята, делаете. Они должны удостовериться, что вы не выполняете кучу работ, которые не приносят никакой пользы, или почему приоритетным является работа без билетов. В этом нет никакого вреда. В идеальном мире это принесет пользу, потому что ваш менеджер может устанавливать ожидания в бизнесе, чтобы вы могли выполнять всю эту работу без давления или перерыва.
Это становится проблемой только в том случае, если ваш менеджер считает, что следует выполнять только работу с билетами, и исключает необходимость технической работы по очистке билетов. Всегда есть технический долг для очистки. Всегда нужно что-то подправить, потому что вы должны, даже если они не дают четкой, немедленной выгоды для бизнеса.
источник
Если стилистическое исправление является частью билета, над которым вы работаете, и оно связано, то нет ничего плохого в том, чтобы зарегистрировать его отдельно с тем же номером билета, с которым вы работали, для лучшей идентификации.
Если вы только обнаруживаете изменения, которые необходимо внести, и они не связаны с заявкой, над которой вы в данный момент работаете, я бы предложил оформить заявки, связанные с техническими долгами, и поместить их в свой журнал ожидания для последующей перефразировки.
Во время планирования вы можете просмотреть билеты, связанные с техническим долгом, и прикрепить их к фактическим билетам на техническое обслуживание, которые вы планируете работать таким образом, чтобы сделать их более связанными.
Это поможет вам устранить эти «из ниоткуда» исправления и сохранить все в секрете под категорией конкретных проблем / заявок, над которыми вы работаете.
источник
Это тоже меня беспокоит (без каламбура :).
Самые полезные проверки содержат комментарии, которые уникальны и специфичны не только для того, что изменилось, но и почему. Я иногда заканчиваю тем, что помещал комментарии параграфа.
Я иногда попадаю в ситуации, когда требование UI отскакивает назад и вперед после начала разработки, требуя, чтобы я также существенно отклонял код (да, иногда реальный мир - отстой). В этих случаях я считаю еще более важным, чтобы мои комментарии давали ясность относительно причины отказа и, что более важно, почему код заканчивается там, где он есть. Затем, когда руководство возвращается после разговора с клиентами и хочет знать, почему, я могу показать им, не вспоминая весь опыт по очереди.
источник