10 советов успешного code review

[@AndroidBroadcast]

🔗 Avito.Tech avito.tech/ 💰 Поддержать проект на Boosty bit.ly/3sratqQ или Patreon patreon.com/android_broadcast 🔗 Telegram канал "Android Broadcast" ttttt.me/android_broadcast

[@ksz87]

Качество кода - это дело наживное. С помощью интернета, менторов и дисциплинированности можно многому научиться. А вот аккуратность, внимательность, отношение к коллегам, которые с твоим кодом будут работать - это воспитывается гораздо сложнее (и вообще часто от рождения в человеке). Поэтому в коллегах и кандидатах я это ценю больше, чем знание каких-то библиотек.

[@beeline09]

Кирилл, все супер! Спасибо! Но джитхаб...режет слух вот уже который выпуск))

[@alekseyyakovlev6924]

Поддержу. Произносится Гитхаб. Можно даже послушать видео от самих разработчиков kzbin.info/www/bejne/rWTNfX2LbKmHa6c

[@AndroidBroadcast]

Я походу от волнения всегда. Извините ((( Буду стараться говорить гитхаб

[@AndroidBroadcast]

А вот тут уже специально говорится

[@davitzakaryan1872]

Спасибо за видео! Давно ждал такого контента!

[@SuperMarkwel]

как всегда по-царски, спасибо

[@dmitrii_leonov]

Классный выпуск. Подметил несколько вещей для себя.

[@fliker2309]

Спасибо за видео!)

[@MarsIanin06]

Спасибо за видео! Интересно будет также узнать о рекомендациях по ревью пулл реквестов от своих коллег.

[@AndroidBroadcast]

Подумаю, но вот сходу рассказать нечего

[@smekalisty]

Спасибо за видос Согласен со всеми пунтками но не про длину строк в 120 и вертикальную черту. Кто програмирует на мониках должны уважать тех кто на ноутах. А в обратную сторону как же. Зачем нам такое пустое незанятое пространство справа. Тут хорошо бы иметь какую-то статискику по диагоналям для разработчиков. Но как по мне сейчас это может быть уже 140-150 символов. Так же работая за дектопом и с мышкой мне проше нажать шифт и проскролить колесико чем пытаться всмотреться в нечитаемые прямоугольники где тут у автора заканчиваются аргументы конструктора где двоеточие и потом где начинается классы наследования. Так по памяти вроде спорим мы за объявление класса ибо в андройде именно тут обычно наблюдается хорошая такая длина. Просто как-то ты уж слишком катигорично выразился Еще хочу добавить что когда ревьювил тестовые задание когда в readme было сразу написанно что-то типа lint: 0 ошибок, или там скриншот с грамматическими ошибками только прям сразу + парню потому что мне уже не нужно запускать анализатор

[@AndroidBroadcast]

Читать код больше 120 символов - сложно. Посмотрите даже правила печатников

[@СергейПанов-з3ц]

Лайк поставил, но комментарий лень писать.

[@bolnoi8]

Я честно не специально, у меня во всех проектах стоит wrap на 120, как получилось, что при создании нового установилось значение в 150 ума не приложу.

[@AndroidBroadcast]

Да это не было лично претензий, а просто хороший стиль. Твой проект напомнил и ты не один такой