Strict commit messages policy

[shmygol]

Предлагаю запретить коммиты, не привязанные к issue. Это конечно жестковатое полиси, так что будем применять только если никто не против.

У меня есть local hook, который проверяет сообщений на следующий паттерн: ^(issue|close|fix|resolve) #(\d+)

[averrin]

Я вообще за. Залей хук в транк и опиши инсталляцию (желательно однострочником) - и будем пользоваться. Не знаю на счет запрещать, но крайне рекомендуемо - точно.

[shmygol]

Надо бы создать папочку для скриптов различных. deployment скрипт туда тоже положить, но в целях безопасности вызывать его не от туда, конечно.

[mvasilkov]

А зачем? Звучит как проект Путина-Мизулиной, давайте что-нибудь запретим просто так.

[shmygol]

@mvasilkov так ведь не просто так. Это очень помогает контролировать коммиты. У меня в продакшене на всех проектах так.

[shmygol]

@mvasilkov вот цитаты двух сообщений из #29 где я объяснял свою поизцию:

trilliput:

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

trilliput:

Вдруг ты поломал какой-то другой функционал? Возникают вопросы: а зачем ты это делал? Что ты фиксил? Как это можно по-другому пофиксить?

[mvasilkov]

Проектная астронавтика. Не хватает еще аналога PEP8 для сообщений в коммитах.

Возникают вопросы: а зачем ты это делал?

Это написано в сообщении к коммиту.

[shmygol]

Возникают вопросы: а зачем ты это делал?

Это написано в сообщении к коммиту.

Из моего опыта информации в коммите обычно недостаточно, так как в задаче прописаны все связи с другими задачами (которые могу изменяться в отличие от сообщения в коммите), комментарии, дата создания, автор (если он отличается от того кто коммитит) и т.д.

[shmygol]

Вообще после вот этого #42 обсуждения совершенно пропало желание в этом участвовать, так что делайте без меня, I'm out.

@mvasilkov А ты чего расстроился-то так? Никто этого внедрядь не будет без твоего личного согласия. В описании ясно написано:

Это конечно жестковатое полиси, так что будем применять только если никто не против.

[mvasilkov]

Ну, я уже не против.

[shmygol]

Ну если это послужило причиной твоего ухода, то ты против, причем кардинально. И это нормально быть против чего-то, но уходить при первом же неугодном предложении странно.

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

[shmygol]

Закрываю пока, чтобы не маячило. Действительно жесткие правила могут отпугнуть кого-то от проекта.