atls / nestjs

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

Обновление зависимостей `signed-url` #269

Closed SlumberyDude closed 4 weeks ago

SlumberyDude commented 1 year ago

С чем связан запрос на фичу?

Обновление зависимостей в модуле signed-url

Расскажите как вы это себе видите

Обновить и зафиксировать версии пакета @atlantis-lab/nestjs-signed-url до максимально возможных таким образом, чтобы ничего не ломалось, а именно проходили команды критерия готовности.

Исключения:

Зависимости typescript, typeorm обновлять не нужно.

Если ломаются тесты и это связанно с переходом NestJs с версии 8 на 10, и проблема не на поверхности (не фиксится заменой в тесте, например, метода .asyncListen на .listen), то пакеты, начинающиеся с @nestjs/... обновлять не выше мажорной 8.

Definition of done (критерий готовности)

Проходят без ошибок команды:

Если в пакете есть скрипты build и prepack, то они тоже должны проходить без ошибок.

Приложите пример реализаций

https://classic.yarnpkg.com/lang/en/docs/cli/workspace/ yarn workspace <workspace> add <package@1.2.3> <keys>

Приложите материалы задачи

Можете посмотреть закрытые PR и issue по теме обновления зависимостей

ArtemMakhaydinov commented 1 year ago

@SlumberyDude В этом проекте "typescript": "4.1.3" В корневом package.json "typescript": "4.5.5" в resolution В папке проекта подогнать TS под залоченную версию?

SlumberyDude commented 1 year ago

@ArtemMakhaydinov Если не будет ошибок то да, можно повысить до 4.5.5. Ну вроде их быть не должно, залоченная ведь работает