atls / nestjs

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

Обновление зависимостей `keto` #266

Closed SlumberyDude closed 8 months ago

SlumberyDude commented 1 year ago

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

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

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

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

Исключения:

Зависимости 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 по теме обновления зависимостей

IgorSmugalov commented 1 year ago

@SlumberyDude Пакет @oryd/keto-client обновился, но он Deprecated , актуаленый @ory/keto-client имеет другой api, поискал что-то по этому поводу в документации - ничего не нашел

SlumberyDude commented 1 year ago

@IgorSmugalov Думаю на новый api точно не надо переходить в рамках обновления зависимостей. Если с deprecated последней версии все работает, то ок.

IgorSmugalov commented 1 year ago

@SlumberyDude

Следующие тесты проходят без ошибок:

Примечания:

Дальнейшие действия?

SlumberyDude commented 1 year ago

@IgorSmugalov Сделай PR. Назови ветку в соответствии с конвенцией, то есть что-то типа feat/keto-update-deps. Потом если в PR все чеки пройдут запрашивай у меня ревью