atls / nestjs

NestJS Custom Workshop
BSD 3-Clause "New" or "Revised" License
33 stars 8 forks source link

Update typesense dependencies #283

Closed RimidalU closed 1 year ago

RimidalU commented 1 year ago

Updated to the latest dependencies in the typeorm-seeding. Exception:

RimidalU commented 1 year ago

@TorinAsakura Видно я не осознал всю глубину глубин rebase(((

Исходные данные:

TorinAsakura commented 1 year ago

@RimidalU Так, ну давай разбираться:

  1. Дублированные коммиты: Если ты делаешь git pull с --rebase после ребейза - ясен красен произойдёт дублирование, так как происходит применение твоих коммитов поверх текущего состояния удаленной ветки в итоге получается наслаивание…

  2. Конфликты в .pnp.cjs и yarn.lock: Это лучше просто перезаписывать своими

  3. Тесты не запускаются: о каких конкретно тестах идёт речь?

Чо делаем

  1. Сделай git pull origin master находясь в master.
  2. Переключись на свою ветку и сделай git rebase master.
  3. Разреши конфликты и сделай git add . и git rebase --continue.
  4. После успешного ребейза сделай git push origin имя_твоей_ветки --force-with-lease.

Про --force-with-lease я уже писал ранее - он тупо безопаснее, чем просто --force, потому что не перезапишет изменения, сделанные другими людьми

Про форс-пуши

Форс-пуш — это хреновая практика, но она приемлема в ветках, которые не являются общими (master, stage,development). Если ты единственный, кто работает в этой ветке, и ты точно знаешь, что делаешь, форс-пуш допустим, но, я бы рекомендовал вообще использовать его как последнюю меру…

RimidalU commented 1 year ago

@TorinAsakura , я всё-таки его осилил)))