ВОобще для меня практика "комменты оставил реджектнул и все" - кажется странной) Если мелочь - ну ок... Но мелочь и быстро фиксится. А если вижу слишком много мелких проблем или какое-то непонимание подходов и инструментов - сразу в личку или даже созвон.
Тем, что это не приводит к пониманию причин, по которым человек не прочел гайд. Может он его не понял, может решил, что гайд не обязателен - да дофига поводов. И нужно бы разобраться в этих причинах.
Низкая эффективность в процессах, если осознать не получается, то объяснять не стоит. Время все сделает само. Типа: научить не могу, поэтому сижу проверяю 90% кода, ну чо эксперт
Работа, конечно. Поэтому нафиг мне человек, которые не умеет поддерживать гайдлайны и требует постоянной проверки. Если дело в процессах онбординга - подправим процессы. Если дело в человеке - нафиг этого человека.
Нет. не нафиг человека. У вас в команде могут быть люди разных бекграундов, разных подходов и так далее. Единственный способ им обучиться и raise the bar - это ревьювить код друга друга. А еще лучше когда тех лиды или лиды домейна это делают.