Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Closed
SlumberyDude opened this issue Aug 25, 2023 · 2 comments · Fixed by #326
Closed

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

SlumberyDude opened this issue Aug 25, 2023 · 2 comments · Fixed by #326
Assignees

Comments

@SlumberyDude
Copy link
Contributor

SlumberyDude commented Aug 25, 2023

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

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

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

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

Исключения:

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

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

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

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

  • yarn install
  • yarn check
  • yarn test unit

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

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

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

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

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

@ArtemMakhaydinov
Copy link
Contributor

ArtemMakhaydinov commented Sep 15, 2023

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

@SlumberyDude
Copy link
Contributor Author

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants