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

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

Closed
SlumberyDude opened this issue Aug 25, 2023 · 4 comments · Fixed by #280
Closed

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

SlumberyDude opened this issue Aug 25, 2023 · 4 comments · Fixed by #280
Assignees

Comments

@SlumberyDude
Copy link
Contributor

SlumberyDude commented Aug 25, 2023

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

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

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

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

Исключения:

Зависимости 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 <[email protected]> <keys>

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

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

@IgorSmugalov
Copy link

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

@SlumberyDude
Copy link
Contributor Author

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

@IgorSmugalov
Copy link

IgorSmugalov commented Sep 13, 2023

@SlumberyDude

  • Все пакеты обновлены до последних версий
  • Типизировал Metadata для KetoAccessControlGuard: без этого не проходили тайпчеки при переходе на nestjs^10

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

  • yarn install
  • yarn check
  • yarn test unit
  • build
  • prepack

Примечания:

  • apollo-server-core - обновлен, но в статусе deprecated

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

@SlumberyDude
Copy link
Contributor Author

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

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