170 похожих чатов

Привет. В проект собираемся вводить код ревью и есть пару

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

3 ответов

12 просмотров

Ну задачи на 8 часов делать

коммит должен быть минимальным, на сколько это возможно. В одном коммите не должно быть "решений" разных задач. Рефактор отдельно, коммит по таске отдельно, добавил логи - отдельно, исправил/добавил комментарий - отдельно. По фичам коммиты тоже минимальные, например, написать сервис хуйни, так вот, каждый метод коммитить отдельно, добавил работу с базой - отдельно. Мерж делать всем табором 😁

Похожие вопросы

Обсуждают сегодня

30500 за редактор? )
Владимир
47
а через ESC-код ?
Alexey Kulakov
29
Чёт не понял, я ж правильной функцией воспользовался чтобы вывести отладочную информацию? но что-то она не ловится
notme
18
У меня есть функция где происходит это: write_bit(buffer, 1); write_bit(buffer, 0); write_bit(buffer, 1); write_bit(buffer, 1); write_bit(buffer, 1); w...
~
14
Недавно Google Project Zero нашёл багу в SQLite с помощью LLM, о чём достаточно было шумно в определённых интернетах, которые сопровождались рассказами, что скоро всех "ибешни...
Alex Sherbakov
5
Ребят в СИ можно реализовать ООП?
Николай
33
Как передать управляющий символ в открытую через CreateProcess консоль? Собсна, есть процедура: procedure TRedirectThread.WriteData(Data: OEMString); var Written: Cardinal;...
Serjone
6
в JclConsole объявлено так: function CtrlHandler(CtrlType: DWORD): BOOL; stdcall; - где ваше объявление с stdcall? у вас на картинке нет stdcall
Karagy
8
https://github.com/erlang/otp/blob/OTP-27.1/lib/kernel/src/logger_h_common.erl#L174 https://github.com/erlang/otp/blob/OTP-27.1/lib/kernel/src/logger_olp.erl#L76 15 лет назад...
Maksim Lapshin
20
~ 2m21s  nix shell github:nixos/nixpkgs#stack ~  stack ghc -- --version error: … while calling the 'derivationStrict' builtin at /builtin/derivation.nix:...
Rebuild your mind.
6
Карта сайта