Код ревью обычно ведется по каким то критериям качества (это нечто большее, чем код стайл). У вашего нового тл-а эти критерии жестче, чем у предыдущего. Обычно это хорошо. Это значит, что ему не нас*ать.
Потребуйте у нового тл-а список критериев оценки. Внимательно ознакомьтесь, если что-то не ясно, или спорно - задавайте вопрос: wtf?
Бывают спорные критерия стиля оформления, их принимают для унификации.
А бывают требования не очевидные, но вполне решающие какую-то свою задачу.
UPD
Вот вам пример критерия, об который сломана куча копий:
Валидация на тип и граничные значения должна быть в каждом публичном/защищенном методе.
Это очень геморный критерий, множество методов состоят из проверки аргументов более, чем на 50%. Но этот же критерий убирает целый класс ошибок + очень помогает в поиске ошибок.
Доходит до того, что он переделывает какие-то названия переменных или классов
Именование классов для вас вкусовщина?))
Или же он недоволен, когда я начинаю править что-то в коде для своей задачи
Вы нарушаете при этом принципы типа SOLID/YAGNI/... ?
Как вы считаете, что здесь играет роль?
Роль играет то, что вы хотите адаптировать под себя человека с на много большим опытом, а не научиться у него.
Просто такой попался лид?
Самая большая похвала, которую я в свое время получил за свой код: "твой код не совсем говно", это при том, что ТЛ - с большой буквы, и мой код от его кода очень тяжело отличить, даже проработав более года на проекте.
Если ваш язык php - можете ознакомиться с требованиями:
https://github.com/index0h/php-conventionsИли же у нас с ним пропасть в навыках, поэтому его коробит от любых моих действий?
Дык учитесь у него!
Сталкивались ли вы с подобной ситуацией? Как бы поступили?
Сталкивался и очень много спорил, по поводу каждого требования, у нас даже стоп слово было "иди на ***" - оно применялось, когда у сторон не хватает аргументации, что бы прийти к конкретному соглашению.
На данный момент, я хочу понять, стоит ли мне просить перевода на другой проект и сколько дать себе времени на то, чтобы привыкнуть к новому лиду?
По моему не стоит переводиться, у вас уже есть человек, у которого можно научиться, попросите его комментировать ошибки, а не исправлять их за вас.
Я понимаю, что я могла бы многое взять у этого человека, но всё сводится к тому (в моем восприятии), что он защищает свой идеальный код от меня.
Еще бы: представьте, вы кардиохирург, делаете операцию и тут подходит мед сестра и начинает тыкать больного скальпелем в ногу, что бы протез было удобно монтировать, хотя нога вполне здорова.
Простой
Простой
Простой
