2016-05-18 2 views
1

Установлено, что каждое отдельное Git-commit должно быть одним логическим изменением. Также установлено, что этот процесс производства часто бывает грязным (см. sausage-making), и его не следует скрывать - защитники Сета Робертсона "Commit often, perfect later".Убирает ли ваш кодекс отдельный Git?

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

  • осветляющие комментарии
  • Изменение порядка функций для улучшения читаемости
  • Коррекция отступы/WHITESPACE
  • Удаление старых тестов как // console.log(variables)

Мой вопрос тогда: является его лучше всего сохранить эти «косметические» изменения в отдельной индивидуальной фиксации? (И если нет, то как они должны быть совершены?)

Отметьте, что я не приглашаю об этом. Разумеется, существует передовая практика; в статье выше подчеркивается enforcing standards, поэтому важно знать, что такое эти стандарты.

Как и статьи Сет Робертсона я прочитал следующие существующие вопросы, но я не могу найти мой вопрос, адресованный в любом месте:

+0

совершить косметические изменения отдельно. Это помогает в проблемной изоляции позже, так как вы обычно можете пропустить всю транзакцию с косметическими изменениями при поиске ошибки. – Gregg

+0

http://stackoverflow.com/a/19656336/6194839 Я нашел это, просматривая ваши ссылки, так как я согласен, что должен быть стандарт. Вот что я хочу, чтобы выделить из этой ссылке: ():

Назначьте из вашей фиксации в качестве косметического средства. –

+1

@BryceDrew Я понимаю, что вы имеете в виду. Ваш комментарий побудил меня найти другие примеры авторитетных руководств по стилю, в которых перечислены различные типы фиксации и рекомендуемый префикс для сообщений о фиксации. Например https://github.com/atom/atom/blob/master/CONTRIBUTING.md#git-commit-messages (префикс «искусство:» для изменений стиля) и https://udacity.github.io/git- styleleguide/(prefix 'style:').Это имеет тенденцию предполагать, что существует конвенция изменений стиля, являющаяся допустимым фиксацией. Я бы добавил эти ссылки в качестве ответа, но, к сожалению, этот вопрос еще не выполнен. – andydavies

ответ

1

Да, косметические изменения должны быть раздельными. Они являются логическим изменением, отделенным от функциональных изменений. Кроме того, упрощается обзор функциональных изменений.