Хотел бы посоветоваться со знатоками, правильное ли у меня отношение к кодревью на новой работе. :) Работаю только месяц на новом месте, заметил, что качество кода в целом в комманде весьма посредственное. И кодревью проходит очень лайтово, никто особо не коментит. Я решил проявить инициативу и покоментил код сначала одного разработчика, который устроился немного позже меня. Комменты были из разряда: чекать параметры публичных методов, корректно именовать методы и переменные (там прям не смог мимо пройти). Он отнесся адекватно и большую часть исправил.
Сегодня комментил код главного разработчика, который меня онбордит. Там таких проблем нет, все в целом ок, но я заметил одно существенное нарушение использования паттерна и его одноименной либы. Описал проблему, предложил решение. В ответ получил весьма в неприятной форме ответ, что так как он написал, делать правильней. Я ответил, что про такой подход вообще не слышал и даже нагуглить не смог, зато есть конкретная рекомендация автора этой библиотеки, как с ней не работать, и привел цитату. Ответа не последовало. А сейчас захожу в битбакет и думаю, либо мой коммент удалили, либо я его не отправил, но есть ощущение, что точно отправлял. Интересно, может ли кто-то в битбакете удалить чужой коммент? :)
Вопрос вот в чем, может ли мне навредить подобная инициатива? Ведь вещи я предлагаю реально стоящие с пруфами и ссылками на источники. Что это за отписки такие: "я бы предпочел оставить как есть", "если есть договоренность такая, тогда исправлю". В компании, где я работал до этого, одного такого разраба просто уволили через месяц, т.к. он не стремился к качеству кода и также отписывался. Из практики, как лучше, продавливать и стоять на своем, если реально предлагаешь улучшить код с минимальными затратами или, может, высказаться и не настаивать на исправлениях, или вообще забить? Просто у меня забить точно не получится, т.к. я привык к высоким стандартам и на собесе мне говорили о высоких стандартах в их компании, а по факту, код пишут кто как привык да еще и не прислушиваются к конструктивным замечаниям. Что думаете? Может, примеры из практики?