История изменений

Есть какие-то оправдания первого варианта?

Многие люди почему-то не заботятся о том, чтобы привести стиль кода в соответствие. Или делают исправление, которое меняет поведение так, что ломается другой use-case. Или предлагают «патчи», которые заново сделать проще, чем проверить. Или предлагают откровенно глючный код. Или решение делается немного не так, как сделал бы основной автор (так как автору на код смотреть придётся гораздо дольше, чем контрибьютеру, код должен выглядеть приятно. Автору).

И как агитировать на применение исключительно второго варианта?

Тут надо среди контрибьютеров разъяснительную работу производить. Но всё больше и больше я убеждаюсь в том, что многие люди просто не в состоянии понять, что же в их патчах не так.

Есть какие-то оправдания первого варианта?

Многие люди почему-то не заботятся о том, чтобы привести стиль кода в соответствие. Или делают исправление, которое меняет поведение так, что ломается другой use-case. Или предлагают «патчи», которые заново сделать проще, чем проверить. Или предлагают откровенно глючный код.

И как агитировать на применение исключительно второго варианта?

Тут надо среди контрибьютеров разъяснительную работу производить. Но всё больше и больше я убеждаюсь в том, что многие люди просто не в состоянии понять, что же в их патчах не так.