Comments 8
А где любопытные предложения и мысли ? Не нашёл )
Если я понимаю, что мр становится большим, я переименовываю ветку в task/draft, создаю от main новую task/main и небольшими, связанными мр, отдаю код на код-ревью. А дальше фиксы идут отдельным мр в task/main после QA
Попробуйте найти для код-ревью комфортное время и подходящую окружающую обстановку. Может быть, вам удобнее проводить код-ревью не за рабочим столом в первой половине дня, а расслабленно лежать на диване после сытного обеда?
у вас код-ревью сложнее собственных задач по проекту?
Добрый вечер! Если кратко отвечать - зависит от человека)
А если более подробно... Есть такая поведенческая модель людей под названием DISC, попытаюсь через неё пояснить. Если сжато передать смысл (думаю, в случае ошибки меня в комментариях поправят) - есть "интроверты", есть "экстраверты". И есть люди, для которых на первом месте чувства и эмоции как свои, так и других людей, и есть те, для которых более важен результат. Далее комбинаторика, два на два, в итоге 4 психотипа получается.
Так вот в основании всего этого деления вроде как не просто ИМХО, а научные исследования, говорящие что интроверты не просто так стараются не лезть в шумные компании с незнакомыми людьми, а экстраверты - наоборот. А потому, что мозг работает таким образом: одни получают серотонин, когда находятся в центре внимания, другие - когда либо отдыхают в одиночестве, либо в компании с близкими людьми. И точно так же с эмпатией и нацеленностью на результат - одним важны чувства и эмоции, другим - следование правилам и эффективность, тоже потому что так голова устроена.
И вот теперь, приняв за правду такую модель, кому будет приятно копаться в чужом коде, встречать наверняка очевидные косяки других людей, и регулярно доказывать коллегам, что они не правы? Как будто только экстравертам, ориентированным на задачи. А вот у интроверта, оринтированного на результат, скорее всего бомбить будет от непонимания у других людей, как ему кажется, очевидных вещей. А если еще и какие-то договоренности нарушаются, которые вроде как не сложно соблюсти - то вдвойне) А интероверты вообще все эти терки в гробу видали.
А вот если просто делать задачу, тут практически всем хорошо при любом раскладе (с учетом того, что мы все же все разработчики и какой-никакой код писать да должны). Хочешь меньше взаимодействия с другими людьми - общайся только по рабочим вопросам, хочешь больше - найди единомышленника и обменивайся мыслями с ним в течение дня. А если еще задача хорошо проработана да спроектирована - то вообще думать можно минимально, остается просто сидеть да работать.
Но только в голове стоит держать, что в этой картине "код ревью" - это не просто проглядеть нейминг да соблюдение архитектурных принципов по диагонали за 5 минут, затрекав полчаса, параллельно попить чаек и сгоняв на перекур)
Вопрос звучал так: «Что мешает приступить вам к код-ревью?»
Один раз собираетесь всей командой и принимаете решение: код-ревью приоритетнее всех остальных задач, кроме починки критического бага, который напрямую уничтожает пользователей.
На это потребуется 10 минут, и можно будет сэкономить вагон времени на написание следующей заметки.
Если человек находит любую причину, чтобы не сделать код-ревью — это вредоносный джун, и его нужно гнать из команды вчера.
Выжимаем максимум из код-ревью и спасаем нервные клетки