Чудны дела твои, господи

[spamsink]

Прислали мне тут из Индии несколько строчек кода, изменяющие логику работы программы, на утверждение.

Я пишу в ответ: "В чём заключается нужда в этом изменении? Приведите тестовый пример."

Через недолгое время: Review #NNN has been cancelled. No further activity can take place."

Что это было, Холмс?

Comments

[vit_r]

Программист уволился.

[spamsink]

Не из Японии, из Индии. :)

[dijifi]

Мне кажется произошло что-то вроде этого

https://youtu.be/6FU7igmYt-o

[spamsink]

Вот я и говорю "Не из Японии, из Индии".

[dijifi]

Тогда сделали менеджером.

[vak]

Инициатива снизу.

Ты будешь смеяться, но мне тоже вчера пришёл странный pull request из Индии. Изменение версии кода в моём проекте в файле VERSION.in. Была версия 0.4.5, стала 0.6.1. Чувак из нашей конторы, но понятия не имею, кто такой. Чего ему нужно, ума не приложу.

[spamsink]

Чувак из нашей конторы

А что, были варианты? :)

Можно предположить нормализацию номеров версий, предложенную каким-нибудь произвольным менеджером.

[vak]

Да чёрт знает, мог быть контрактор какой.
Если кто из менеджеров внезапно займётся нормализацией версий, сначала спросит у меня.
Настораживает, что "VERSION.in edited online with Bitbucket".
Может, новенький программер просто осваивает битбакет.

[juan_gandhi]

Да испугались просто.

[spamsink]

Это объясняет сброс review, но не объясняет собственно запрос.

[juan_gandhi]

Джун послал запрос без спросу, ну или сеньор не имел времени утвердить. Из Америки пришел суровый отпор. Джуну вежливо объявили взыскание. Джун уписался от страху. Индия ж.

[spamsink]

Не то чтобы отпор был сильно суровым. Или просить тесты среди определенного контингента считается хуже авады кедавры?

[juan_gandhi]

У определенного контингента насчет тестов всегда есть отмазка: "нам сейчас некогда". Они не ждут вопроса типа "are you a professional?", и, если услышат, шибко обидятся. Тут все зависит от менеджмента. А ты им, индийским инженерам, не менеджер, как я понимаю. Выполнять твои хоть приказы, хоть пожелания, у них нет никакой мотивации.

[spamsink]

Причём я просил даже не теста в nightly test set, а просто примера, демонстрирующего нужду в изменении - типа, ради чего, вообще, весь сыр-бор.

[sab123]

Я как-то работал с человеком (русским), который любил делать мелкие бессмыссленные изменения. Типа, как он говорил, для уменьшения engineering debt.

[dijifi]

Жизнь учит по другому. Мой любимый багфикс — убрать лишний нолик из кода драйвера который до этого не менялся пятнадцать лет. GPU exception в довольно популярной игре пофиксено!

[spamsink]

Это было мелкое осмысленное изменение!

[dijifi]

Что я хотел сказать — практика и пинки коллег быстро отбивают жгучее желание поменять все табуляции в пробелы или наоборот. Вот такое изменение становится идеалом.

[spamsink]

Намерение хорошее, а как он обосновывал - если обосновывал - что этими изменениями достигается результат?

[sab123]

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

[dijifi]

Один интерн был поставлен что-то добавить к какой-то тестовой программе, написанной в стиле абстрактных типов данных — с переопределением операторов умножения для матриц и т.п., и он возмутился до такой степени, что послал сотням программистов сообщение, что так делать нельзя, потому что так делать нельзя никогда!!! Видно надо благодарить профессора, так как интерны обычно своих мнений сформировать ещё не успевают.

[spamsink]

И все бы ничего, если бы после этих изменений не ломался мерджинг для десятка сидящих в кодревюе готовых диффов.

Ой, не говори!

[larisaka]

Нет, ну бывает же? что-то делаешь, делаешь, делаешь, а потом вдруг глянешь с другой стороны, тьфу! Зачем, не надо было это все.
Ну не знаю, у меня бывает. Может, человек нормальный и просто додумался с наводки-то.

[spamsink]

Похоже на то.