Разбор задач с WO / Код здорового человека

Привет, username 👋

Если ты тут, то это означает, что тебя пригласили на YAMP.

Этот репозиторий яляется примером задачи для одной из активностей митапа. Данной активностью является публичный разбор задачи с нашего прошлого Weekend offer.

Суть задачи

Задача заключается в том, чтобы провести максимально качественный Code review. Именно Code review, а не переписать все правильно.

Что такое качественный Code review?

  1. Нужно найти все проблемы в коде
  2. В комментарии к проблеме нужно описать почему это проблема
  3. Если есть советы о том, как лучше сделать, это нужно указать. Но не нужно писать прямо код.

PS

Чтобы было проще представить что это за Pull request, мы придумали легенду:

Представь, что ты навставник iOS стажера. Он целую неделю потратил на то, чтобы сделать последнюю задачу, которую ты ему дал. И вот в пятницу он залил свой PR. Тебе нужно, во чтобы то ни стало, успеть сделать ревью именно сегодня и сделать его макисмально качественно, тк завтра ты улетаешь в отпуск и будешь без связи. Если не сделать ревью быстро, то стажер будет ждать его весь твой отпуск. Если сделать его не качественно, то стажер не сможет все исправить, а связаться он с тобой уже не сможет. Кроме того, ты не хочешь писать код за стажера, а хочешь его чему-то да научить, поэтому даешь только рекомендации, а не переписываешь код.

Вот такие вот дела

glhf