Информация об изменениях

Сообщение Re: Как решать конфликтные ситуации при code review? от 30.01.2024 17:26

Изменено 30.01.2024 17:35 novitk

Re: Как решать конфликтные ситуации при code review?
Здравствуйте, Aleksei_Lekomtsev, Вы писали:

A_L>Часто бывают спорные ситуации во время code review. Это либо опыт у людей разный, либо дело вкуса, какие-то "холиварные" темы

A_L>Например, писать/не писать комментарии, название переменных, использовать/не использовать var и т.д.. Список можно продолжать до бесконечности
A_L>Это те моменты, которые никак качественно не влияют на работу приложения. Т.е. например, это не о том как код используется больше/меньше ресурсов

Если речь идет о стилистических моментах, то спор имеет смысл только если обсуждаются именно они, а не конкретный pull request.
Если в команде они не формализован никак, то стоит конфликтные моменты расписать в wiki. Ты возможно не будешь с ними согласен, но это обычно "not the hill do die on". Извиняюсь за англискую поговорку, но на ум ничего рускоязычного не приходит.
Re: Как решать конфликтные ситуации при code review?
Здравствуйте, Aleksei_Lekomtsev, Вы писали:

A_L>Часто бывают спорные ситуации во время code review. Это либо опыт у людей разный, либо дело вкуса, какие-то "холиварные" темы

A_L>Например, писать/не писать комментарии, название переменных, использовать/не использовать var и т.д.. Список можно продолжать до бесконечности
A_L>Это те моменты, которые никак качественно не влияют на работу приложения. Т.е. например, это не о том как код используется больше/меньше ресурсов

Если речь идет о стилистических моментах, то спор имеет смысл только если обсуждаются именно они, а не конкретный pull request.
Если в команде они не формализован никак, то стоит конфликтные моменты расписать в wiki. Ты возможно не будешь с ними согласен, но это обычно "not а hill to die on". Извиняюсь за англискую поговорку, но на ум ничего рускоязычного не приходит.