Ревью

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

В репозиторий добавлен файл CODEOWNERS со списками мейнтейнеров проекта. Ревьюеры из этого файла автоматически проставятся у PR и получат уведомление о новом PR.

Правила ревью:

• К мержу допускаются PR, получившие не менее 2 аппрувов.
• При ревью происходит распространение новых знаний о проекте и используемых практиках. Можно обсуждать непонятные моменты, задавать вопросы, предлагать улучшения и иные варианты решения задачи.
• Важно соблюдать вежливость в обсуждении вопросов и разногласий.
• При разногласиях рекомендуем созвониться и обсудить вопрос голосом. Так быстрее. После напишите в комментарии к PR итог созвона, чтобы остальные ревьюеры смогли узнать про ваши договоренности.

Помните:

• Читать чужой код всегда сложнее, чем писать свой
• При работе над своей задачей очень легко забыть о чем-то важном, особенно если она объемная
• Ревью может занимать много времени, это нормально
• Ревьюеры не ставят аппрувы просто так, если им непонятна сделанная в PR работа
• Ревью – это процесс общения, и часто какие-то пункты можно опустить по договоренности.