Думаю, мне действительно нужна помощь, чтобы помочь людям, которые забывают, круиз-контроль рассылает подробности коммитов всем, поэтому люди стремятся добавить содержательный комментарий, но иногда забывают.
Можно ли сделать комментарии CVS обязательными для коммитов?
comment
Это довольно бесполезно, ваше время лучше потратить на то, чтобы убедить людей написать осмысленные комментарии. В противном случае вы просто получите комментарии с 5 пробелами или с надписью «Черт Дэн».
- person Vinko Vrsalovic schedule 11.11.2008
comment
@Chris: CVS иногда используется как общий термин (стоит проверить) ;-)
- person DilbertDave schedule 11.11.2008
Ответы (5)
Вы можете написать перехватчик перед фиксацией, который проверяет это. Для этого требуется доступ к серверу CVS.
person
Carl Seleborg
schedule
11.11.2008
Вы также захотите, чтобы человек следил за комментариями, которые добавляют люди.
Или ожидайте шквал комментариев «foo», «checkin», «commit», «changed code», когда люди будут пытаться обойти вашу проверку.
В прошлой жизни мы также навязывали упоминание идентификатора дефекта/задачи и проверяли наличие идентификатора в системе отслеживания задач...
person
The Archetypal Paul
schedule
11.11.2008
исправлена ошибка - мой личный фаворит
- person tloach; 11.11.2008
новые вещи иногда так удобно :P
- person David Santamaria; 11.11.2008
добавленная функция - это та, которую я использовал несколько раз сам.
- person Chris Charabaruk; 11.11.2008
@David: я бы больше волновался, если бы увидел старые вещи... ;)
- person tloach; 11.11.2008
Вместо того, чтобы навязывать комментарии через CVS, просто сообщите разработчикам, что вы можете проводить выборочную проверку комментариев, поэтому, пожалуйста, добавляйте полезные комментарии при регистрации кода. Хотя я согласен с тем, что комментарии в самом коде более полезны, иногда полезно знать, какова была цель проверки кода (например, исправлена ошибка округления сумм налога в счетах-фактурах). Однако этим можно легко злоупотребить, когда кто-то постоянно проверяет комментарии, что тратит огромное количество времени. Таким образом, это зависит от системы чести и того факта, что никто не хочет, чтобы его выделяли за то, что он не оставил однострочный комментарий, который вы просили.
person
Elie
schedule
11.11.2008
Я думаю, что это все еще стоит применять, а также проводить выборочные проверки. Это мешает людям забыть добавить комментарий вообще.
- person The Archetypal Paul; 11.11.2008
Файл verifymsg выглядит так, как вам нужно, хотя Я никогда не пробовал это сам. Если комментарий фиксации не соответствует определенным критериям, фиксация прерывается.
person
jrb
schedule
11.11.2008
Мы используем Subversion и Tortoise, и это возможно.
В основном вам нужно установить параметр tsvn:logminsize в папке ствола. Это позволит вам установить минимальную длину комментария (все, что > 1, приведет к обязательным комментариям).
person
DilbertDave
schedule
11.11.2008
Во-первых, это решение только для Subversion, а во-вторых, оно работает, только если кто-то использует TortoiseSVN.
- person Chris Charabaruk; 11.11.2008
Как уже говорилось, CVS иногда используется как общий термин (стоит проверить) ;-)
- person DilbertDave; 11.11.2008
