По обе стороны ревью

По обе стороны ревью

Последнее время часто стал сталкиваться со всевозможными ревью, причем, находясь то по одну, то по другую сторону баррикад. Заметил и за собой, и за другими несколько странностей, про которые хотелось бы рассказать.

Странность первая – ревьюеруемый – жертва.

Много раз видел такую ситуацию – человек отдает свои вымученные труды на ревью какому-то умудренному опытом старцу и со страхом ждет его результатов. При этом, получая их практически в любом виде, пусть там будет даже два минорных камента, расстраивается и моментально мрачнеет. Воспринимая результаты ревью как выпад, подвергающий его профессионализм сомнению. Но ведь это совсем не так.

Совет жертве. Каменты всегда останутся всего лишь каментами. Это не прямое руководство к действию, а пожелания. И вы решаете применять каменты или нет, при этом не стоит забывать, что ответственность за задачу лежит всегда на вас. И может все-таки стоит попытаться понять комментарии.

Странность вторая – ревьюер делает одолжение.

Наблюдал как-то такую картину, у человека была задача как можно лучше протестировать и задокументировать функционал. Этот самый человек, придумал все идеально, выполнил красиво, и бежит с горящими глазами к своему старцу седому. А тот, скрепя стулом, повернулся, и покряхтев мрачным голосом сказал «Ок, спасибо, молодец». Отвернулся к монитору и продолжил думать о великих свершениях.

Совет умудренному опытом старцу. Незачем из этого процесса делать каторгу для всех. Не хватает времени? Скажите прямо. Это будет лучше, чем вы будете из себя выдавливать невнятные слова, при этом сверкая синяками под глазами. Не хотите проводить его прямо сейчас? Отложите, по той же причине, но в разумных пределах, вам-то тоже никого нельзя подводить.

Странность третья – ревьюеруемый действует с позиции «Не знаю, так ща придумаю».

Это, наверное, самая частая странность. Проходит ревью, задаются какие-то вопросы, и вдруг вопрос касается того, что мы не учли, или просто неправильно поняли. И тут включается защитная реакция, мыслительный процесс ответчика выглядит примерно так: «Я, блин, про это и забыл», «Тэээкс, счас по шурику что-то придумаю», «А скажу-ка я, что я сделал вот так, вдруг прокатит». Говорит невразумительный ответ, вместо того, чтобы сказать: «Не учел, спасибо, подумаю и поправлю». И этот невразумительный ответ порождает новые вопросы, а потом по накатанной: ревью растягивается, все недовольны, появляется легкий конфликт.

Совет шустрому мыслителю. Никогда не забывайте, что цель у всех одна – это качество продукта, который мы в данном случае производим. И скрывая что-то или попросту «отмазываясь», мы ставим это качество под угрозу. Да и прокачать получится только скилл «как отмазаться», а не тот, который является главной целью для нас.

Странность четвертая – ревьюер слишком крут, чтобы тратить на это время.

Приходит время ревью, забукан митинг рум. Сидит ничем не примечательный парень за столом, на котором лежит листок бумаги, исписанный нервным подчерком. И тут открывается дверь, озаряемый лучами славы и успешности заходит ревьюер. Бросив беглый взгляд на листик, говорит: «Все не то. 17 ошибок. Подумай еще», и выходит под мелодию угасающей надежды того самого паренька.

Совет славному и великому ревьюеру. Ревью – это процесс обоюдовыгодный. Великий ревьюер может стать более великим, отточив еще разок свое мастерство рассказчика, оратора или просто сочувствующего. Хорошо выдавать обратную связь – это тоже большой труд, важный скилл и великое благо. И этому также надо учиться, как и всему остальному.

Странность пятая – общая.

«Прописные истины». Как много они похоронили хороших идей и начинаний. Часто думая, что каждый участник процесса понимает эти самые прописные истины одинаково, мы их просто пропускаем, считая, что не стоит тратить на них время. А потом оказывается, что мы по-разному воспринимали основы, и весь процесс затихает. И то самое ревью не приносит по сути пользы, вроде бы сошлись, вроде бы договорились, но осталось недопонимание, которое потом может вылезти боком.

Совет участникам. Не ленитесь, повторение – мать учения. Любое ревью можно воспринять как любой другой процесс в компании или на проекте, и во время ревью тоже можно работать с рисками. И почему бы, потратив 1-5 минут, проговорив «прописные истины», не снизить риск того, что кто-то что-то трактует по-своему, и поэтому не достигнет желаемого.

А в конце хотелось бы напомнить: не забывайте, что тот, чьи труды вы ревьюите сегодня, может завтра ревьюить ваши.

Test Engineer at Sigma Software. Has around 4 years of experience in testing. Interested in: all testing types, process analysis/improvement, estimating, smart documentation

Leave a reply

Blue Captcha Image
Refresh

*