Size: a a a

2021 December 17

KN

Kirill Nesmeyanov in PHP
Но добавление префиксов, вида [TASK-42] Do something решает эту проблему
источник

КГ

Константин Грачев... in PHP
Удобно всё же когда все детали таски в одном коммите, а не логика в одном, тесты в другом
источник

КГ

Константин Грачев... in PHP
Рефакторингу в этом коммите не место. Как минимум не удобно ревью делать проверяя всё подряд.
Поэтому у нас рефакторинги были отдельно, задачи отдельно
источник

МФ

Максим Федоров... in PHP
источник

МФ

Максим Федоров... in PHP
ну если гитлаб
источник

АС

Альберт Степанцев... in PHP
Удобно уметь смотреть мерж-реквесты, а не заставлять делать дичь.
источник

КГ

Константин Грачев... in PHP
А что есть дичь?
источник

A

Alex in PHP
Ахаха беру на вооружение!
источник

КГ

Константин Грачев... in PHP
Ну кроме сквошить руками когда есть кнопка
источник

АС

Альберт Степанцев... in PHP
Один коммит на таск - дичь.
источник

КГ

Константин Грачев... in PHP
И это дичь потому что ... ?
источник

АС

Альберт Степанцев... in PHP
Потому что вы не умеете в гит.
источник

КГ

Константин Грачев... in PHP
Держите в курсе
источник

АС

Альберт Степанцев... in PHP
У вас и так после мержа будет один мерж-коммит.
источник

АС

Альберт Степанцев... in PHP
Так желаемый вами.
источник

MM

Maksim Masalov in PHP
Ну как минимум потому что в маленьких коммитах удобнее смотреть измененный код и понимать зачем это сделано, чем в одном огромном общем.
источник

A

Alex in PHP
Ок спасибо большое!) У меня теперь отдельный вопрос.

Стоит ли открыть во лбу третий глаз смириться с тем, что ревьюер просит "перекрасить сарай в синий цвет", а ты итак понимаешь что по большому счету это не так важно, главное чтобы твои наработки скорее попали в общую кодовую базу
ИЛИ
пойти по принципиальному пути и каждый раз доказывать человеку что он докапывается до мелочей или что ошибается ссылаясь на примеры в умных книгах и статьях
источник

КГ

Константин Грачев... in PHP
Если у вас есть огромные общие коммиты то у вас явно проблема с декомпозицией задач.
А маленькие уровня "я тут метод добавил" не несут никакой полезной информации.

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

АС

Альберт Степанцев... in PHP
Я бы пошел по третьему пути - ушел бы оттуда.
источник

A

Alex in PHP
Ура я не безумец, я тоже об этом думал :)
источник