Должен ли я совершить косметические изменения? - PullRequest
31 голосов
/ 12 января 2010

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

Что мне делать в следующий раз, когда яприходится исправлять мелкие вещи, такие как:

  • Удаление и сортировка использований (в .NET, импорт в Python, включает в себя C ++)
  • Правильные отступы, интервалы и разрывы строк

Ответы [ 13 ]

0 голосов
/ 12 января 2010

Мне нравится часто совершать. Конечно, в любое время происходят заметные изменения. Так легко. Если вы начнете разделять фрагменты кода и попытаетесь зафиксировать некоторые из них раньше, чем позже, вы в конечном итоге забудете совершить что-то очень важное.

Короче говоря: часто совершайте и ВСЕГДА документируйте изменения. Когда есть ОГРОМНОЕ изменение, пометьте его.

0 голосов
/ 12 января 2010

Передайте им следующее большое изменение как sidenote. По крайней мере, так я и сделаю.

0 голосов
/ 12 января 2010

Эти "мелкие" вещи исправляют? Если да, передайте их. Если нет, не надо.

Действительно, это зависит от того, что вы и ваша команда считаете важным.

...