Генерация корреспондентского счета

[shir]

Я изменил название метода с ks на correspondent_account для соответствия английскому переводу. Но для старого названия создал алиас.

[asiniy]

Алиасы, опять же, не нужны. Сделай это одним коммитом, пожалуйста. Тяжело ориентироваться в трёх коммитах

[shir]

Эмм... Я извиняюсь, но зачем в трех коммитах ориентироваться? Я обычно на вкладке "Files changes" смотрю все изменения. Как будто бы одним коммитом. Там же и комментировать можно по строкам.

[asiniy]

Зачем делать 3 коммита по одной функции? Так история комитов будет мусориться

[shir]

Потому что делал этот функционал раздельно, поэтому и разные коммиты. А что с этой историей потом делать чтоб она такая идеальная была? На выставке что ли показывать? Вот на равном месте проблема. Нигде еще не сталкивался с тем чтоб надо было историю коммитов чистить. Тем более что нормальные названия коммитов, нормально разделена функциональность по коммитам.

[asiniy]

Одна функция - один коммит. Это правило хорошего тона в среде rails разработчиков. Если не знаешь, как сплющить, я подскажу

[shir]

Правило хорошего тона, это разделение по коммитам стабильных или логических состояний. Про одна функция-один коммит это бред какой-то.В данном случае можно рассматривать как была сделана функция. Потом спустя время в нее добавился функционал. Или при каждом изменении функционала, нужно старые коммиты пересортировывать? Бред же.

[shir]

Я переделал историю, т.к. хозяин барин, но подобные требования все равно бред какой-то.

[asiniy]

Нифига не бред. Сначала накоммитят несколько сот раз по строчке, а потом копайся бисектом в этом говне ;)

В любом случае, спасибо