Месяц на первой работе, первый настоящий PR: форма загрузки банковской выписки, 280 строк. Получила 37 комментариев от двух ревьюеров. Половина про стиль, несколько про N+1, пара про то, что я считаю в контроллере то, что должно быть в модели. Всё по делу, понимаю. Но вечером в автобусе до Кудрово сидела и думала, что зря ушла из бухгалтерии. Это нормально для первого PR или я правда слабая?
Нормально. 37 замечаний на первый PR в незнакомом проекте значит, что ревьюеры отнеслись всерьёз и потратили на вас час, а не пролистали. Стиль уйдёт через месяц, когда привыкнете к RuboCop, N+1 через два. Вот если те же замечания будут повторяться в пятом PR, тогда стоит задуматься.
Мой первый PR на Rails после восьми лет на PHP собрал 50 с чем-то комментариев, и я тоже вечером думал, не вернуться ли на Laravel. Не вернулся) Заведите файлик и выписывайте туда повторяющиеся замечания, через месяц начнёте ловить их сами до отправки.
у меня на первом было 22, и я неделю боялся открывать уведомления. так что понимаю
Половина про стиль это работа для rubocop -a перед коммитом, а не для ревьюера. Попросите в команде настроить
Спасибо, отпустило. Завела файлик, как советовал Валерий, там уже 14 пунктов. Второй PR собрал 9 замечаний и ни одного про кавычки