Юдин Павел

[PavelUd]

@dmnovikova

[dmnovikova]

хорошая практика перед return оставлять пустую строку
это повышает читаемость методов

[dmnovikova]

Небольшие замечания по работе с гитом:

• Разбивать изменения на коммиты следует исходя из смысла этих изменений, а не пофайлово, чтобы каждый коммит представлял собой одно небольшое логически завершенное действие (как правило, такой код должен билдиться). Понимание нужного размера приходит с опытом, сейчас нужно постараться их не делать очень большими или очень маленькими.

• Лишние изменения не должны попадать в коммит. Например, не нужно добавлять новый парсер, и при этом рефакторить какой-нибудь легаси в одном коммите. Так и с тестами :) - лучше форматирование и добавление новых случаев разбивать на отдельные коммиты.

• Название коммита должно отражать содержимое — тут все хорошо )

Зачем все это?
Потому что ревьюеру выделить определенную идею среди множества изменений непросто, время, потраченное на ревью увеличивается в несколько раз. Поэтому коммитить пофайлово нет смысла, тк это не решает проблему.
А еще в работе не так уж редко приходится обращаться к истории коммитов, чтобы разобраться в малознакомом коде.