sedmin писал(а) 06. Марта 2008 :: 19:00:Я был в ситуации принимающего такие патчи. И если человек хуже меня ориентируется в проекте, то таким способом можно добавить только какую-нибудь примитивную педальку. Попытка принять что-то более серьезное приводила к 2-4 итерациям, полсле которых было понятно, что проще сделать самому, чем объяснить человеку, как это следует сделать.
Согласен, но тут есть еще два момента.
1. В процессе объяснения ты создаешь себе еще одного разработчика в проекте.
2. Если человек не в состоянии за 4 итерации создать правильный патч, то это серьезная причина 20 раз подумать, прежде чем принимать от него какие-то патчи вообще.
Оба момента являются достаточно ценными для того, чтобы их игнорировать.
sedmin писал(а) 06. Марта 2008 :: 19:00:Ты, Федор, конечно принимал патчи, но согласись, что, например, от Михаила ты принимал их практически не глядя.
Не припомню ни одного патча от Михаила, который бы мне прищлось принимать.
Вот от Лехи Диркса - принимал не глядя. Очень положительное ощущение осталось.
Однако были граждане, "неглядный" прием патчей от которых обернулся потом очень серьезным дискомфортом для меня.
sedmin писал(а) 06. Марта 2008 :: 19:00:Тут дело в том, что даже если патч будет от человека одного с тобой уровня, то у тебя все равно будет свой взгляд на принимаемое решение. А если патч оказывается достаточно сложным, то проще оказывается сказать "ну нехай", чем полностью в нем разобраться, да еще и скорректировать то, что можно было бы поправить.
кмк, это зависит в первую очередь от настроя принимающего патчи. В любом случае, получив (вместо набора патчей) архив с "я вот тут тебе все полностью переработал, посмотри плиз", сказать "ну нехай" хочется гораздо быстрее.
sedmin писал(а) 06. Марта 2008 :: 19:00:А если в патч попадает рефакторинг, то тут вообще - дело труба.
А не надо мешать все в кучу. Один баг - один патч. Чем меньше шаг - тем проще и понятнее патч. На рефакторинг стоит заводить отдельный баг.
sedmin писал(а) 06. Марта 2008 :: 19:00:У меня такое ощущение, что должна сформироваться команда разработчиков, которые доверяют друг другу.
"должна сформироваться"... Никому она ничего не должна. Это задача координатора сделать так, чтобы он доверял разработчикам, а разработчики доверяли координатору и друг другу. И система приемки патчей - это как раз тот механизм, который может помочь всем участникам процесса выяснить: кому стоит доверять, а кому нет.