Вопрос не про код, но болит. В команде появились двое новых ребят, и после пары моих ревью один из них стал отвечать односложно и молча переделывать всё подряд, даже то, что было просто мнением. Поговорили, и выяснилось, что каждый комментарий он воспринимал как приказ. Я пишу коротко, по-рабочему: «тут N+1», «вынеси в метод», и мне казалось, что это нормально.
Сейчас пробую помечать вес замечания: nit: для вкусовщины, которую можно проигнорировать, вопрос:, когда сам не уверен, а без пометки только то, что блокирует мерж. И в начале ревью одной строкой пишу, что в PR хорошо, если там действительно есть что-то хорошее. Какие ещё приёмы у вас реально работают? Не из статей про мягкую обратную связь, а из жизни.
Пометки работают, проверено. Ещё помогает писать не «вынеси в метод», а «я бы вынес в метод, потому что ниже то же самое повторяется». Причина в комментарии превращает приказ в аргумент, с которым можно спорить, и человек начинает спорить, а не молча исполнять.
Помогает раз в неделю пройтись по ревью с новичком голосом. В тексте «зачем так?» читается как наезд, а вслух это просто вопрос
и слово «очевидно» из комментариев убрать. новичку ничего не очевидно