Как стать автором
Обновить
18
0
Сергей Жигалов @Zhigalov

NodeJS разработчик

Отправить сообщение

Где можно приобрести футболки, как у преподавателей? Было бы здорово с гордостью надеть очередную футболку по окончанию курса!

Спасибо за ваш комментарий! Это единственный рецепт который у меня заработал.
image


Не забыть вернуть значение pro5v328.upload.speed=57600 в Arduino/hardware/arduino/boards.txt. Я его менял по совету автора статьи. К сожалению, рецепт автора неизбежно заканчивался


avrdude: stk500_recv(): programmer is not responding
avrdude: stk500_getsync() attempt 1 of 10: not in sync: resp=0x00
Привычка, которая осталась с тех времён, когда у меня ещё не было WebStorm. Скорее всего, все действия, которые я делаю при коммите, можно выполнить в IDE.

Мне хочется все команды для работы с git вызывать в одном месте. Могу ли я в IDE выполнить ребейз с флагом --interactive или лог с ключами --pretty='%h: %B' --first-parent --no-merges --reverse? Если можно, с удовольствием полностью перееду в IDE.

А команду git log чтобы сформировать описание пулл реквеста когда и как используете? Выполняете команду, а потом вручную копируете текст и затем вставляете в GitHub?

Да, выполняю в консоли и копирую вывод в описание к пулреквесту.
А от меня риторический вопрос, как отучить разрабов пушить закомментированный код?

Для JavaScript видел такой экзотический способ. Добавляем правило ESlint, которое разрешает писать комментарии только с большой буквы. При комментировании кода в большинстве случаев первая буква будет маленькой. Таким образом, если закомментировать код то не пройдёт линтинг. В связке с husky это не даст закоммитить.

На практике такой способ не использовал. Это правило ESlint мне кажется необоснованным и слишком строгим.
Перед ребейзом на мастер можно добавить пулл из апстрима. Тогда ты не только сделаешь интерактивный ребейз, но и переложишь свои изменения поверх самой свежей версии основной ветки кода. Чем актуальнее твой код, тем проще его мержить.

Ребейз на себя не пробовал, спасибо за рекомендацию.
Действительно, пользу приносит только пуллреквест целиком.

Разбиение по коммитам это один из способов донести до ревьювера мысль. Рассказать по шагам. Акцентировать внимание на деталях, которые легко пропустить в общем потоке.

Иногда, мысль, которая выражена в пуллреквесте, не укладывается у меня в голове. Она большая и сложная. А большие и сложные задачи я обычно декомпозирую на маленькие и простые. Перенёс эту практику на пулреквесты и мне стало удобнее.
Решение задачи действительно процесс творческий. Бывает в одном коммите реализовываешь функцию, а в следующем её почти полностью переписываешь. Тогда старания ревьювера по просмотру первого коммита сводятся к нулю.

Можно минимизировать негативный эффект, если преред созданием пулреквеста прибраться в коммитах. Когда каждая простая мысль выносится в отдельный коммит, то перед ревью можно коммиты про одно и тоже объединить в один (ребейз + перенос коммита + сквош). Это часть саморевью.

Как и в любом творческом процессе надо найти компромисс

Информация

В рейтинге
Не участвует
Откуда
Екатеринбург, Свердловская обл., Россия
Работает в
Дата рождения
Зарегистрирован
Активность