Здравствуйте, Aleksei_Lekomtsev, Вы писали:

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

Если это непринципиальные моменты типа упомянутых, проще всего использовать code style,
желательно автоматически (набор правил, которые энфорсятся прямо в IDE, чтобы до review эти вопросы даже не доходили)
То есть, чтобы код написанный не по правилам, просто не компилировался. Займись лучше этим чем препирательством.

Здравствуйте, Aleksei_Lekomtsev, Вы писали:

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

Если это непринципиальные моменты типа упомянутых, проще всего использовать code style,
желательно автоматически (набор правил, которые энфорсятся прямо в IDE, чтобы до review эти вопросы даже не доходили). Линтеры, рулесеты, вот это вот.
То есть, чтобы код написанный не по правилам, просто не компилировался. Займись лучше этим чем препирательством.