А как иначе?
Взято из телеграмма - Инкогнито
Как правильно “придираться” во время код ревью
Когда вы проверяете код своего коллеги не нужно требовать от него “идеального” кода.
ℹ️ Правило такое: вам следует апрувить изменения, если они улучшают в целом кодовую базу, даже если они не идеальны, ведь идеального кода не существует. Код можно сделать лучше чем он был до этого, но не идеальным.
Если вы увидели, какие-то небольшие помарки в коде, которые совершенно не критичны, то не нужно категорично требовать их исправления. Вы можете оставить комментарии, и оставить на усмотрение автора, необходимо ли вносить какие-то изменения.
ℹ️ В стандартах код ревью гугла, описано, что в таких ситуациях можно использовать префикс NIT, в комментариях к пул реквесту. NIT — сокращение от “nitpick” или “придираться”.
❓В каких ситуациях стоит использовать NIT
- Чтобы выразить своё мнение или личное предпочтение. Очень важно отличать такие комментарии, от фактов. Довольно часто ревьюеры выставляют собственное мнение как истину и единственно верное решение
- Чтобы указать автору на мелкие замечания, которые не обязательно исправлять
- В целях обучения, когда вы менторите начинающего разработчика и ваш комментарий носит чисто образовательный характер
Подробнее об этом можно почитать здесь - The Standard of Code Review
Сможете найти на картинке цифру среди букв?
Справились? Тогда попробуйте пройти нашу новую игру на внимательность. Приз — награда в профиль на Пикабу: https://pikabu.ru/link/-oD8sjtmAi
ИИ помогает проводить ревью
Телеграм канал для тех, кому интересно, заменит ли искусственный интеллект разработчиков.
Создать чистый код
Начал писать код приложения, на Swift и возник вопрос о чистоте кода. Еще сам код не дописан, но хотел спросить о его рефакторинге. Кто делает рефакторинг для исходника? Это правда так помогает при постоянной основе?
Вопрос «тулза для ревью»
Добрый день!
Встал вопрос о том, что пора бы заиметь централизованную тулзу для ревью, заместо переписок в Аутлуке.
Но есть момент - тулза должна работать с раздельными реквестами без репозитория (не спрашивайте почему, но репозиторий вот сразу вообще нет. Точка.). Txt до, txt после - пул и ревьюеры принимаются за дело.
Bitbucket, github отметаются, потому что они работают с репозиториями.
Пока найден только Review Board, но в нём слишком много недостатков, чтобы останавливаться на этом.
Может у кого ещё есть на примете какие подходящие продукты?