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

Update keto dependencies versions #280

Merged
merged 18 commits into from
Jan 19, 2024

Conversation

IgorSmugalov
Copy link

All dependencies updated to the last versions in the keto module
Except:

  • typescript

@TorinAsakura
Copy link
Member

конфликт

Copy link
Member

@TorinAsakura TorinAsakura left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@IgorSmugalov IgorSmugalov force-pushed the feat/bump-keto-dependencies-versions branch from df643d2 to a7f1c5d Compare September 15, 2023 15:01
@IgorSmugalov

This comment was marked as off-topic.

@SlumberyDude

This comment was marked as off-topic.

@IgorSmugalov

This comment was marked as off-topic.

@ArtemMakhaydinov
Copy link
Contributor

ArtemMakhaydinov commented Sep 18, 2023

@TorinAsakura, @SlumberyDude
Был дополнительно установлен пакет ts-morph, его требовал при обновлении @nestjs/graphql.
Все успешно обновилось за исключением:
"@nestjs/core": "10.1.3"
При повышении чек выдает:

➤ YN0000: │ packages/keto/src/guards/keto-access-control.guard.ts
➤ YN0000: │
➤ YN0000: │ Error: Object is of type 'unknown'.
➤ YN0000: │
➤ YN0000: │
➤ YN0000: │ packages/keto/src/guards/keto-access-control.guard.ts
➤ YN0000: │
➤ YN0000: │ Error: Object is of type 'unknown'.
➤ YN0000: │
➤ YN0000: │
➤ YN0000: │ packages/keto/src/guards/keto-access-control.guard.ts
➤ YN0000: │
➤ YN0000: │ Error: Object is of type 'unknown'.

@SlumberyDude
Copy link
Contributor

@ArtemMakhaydinov Покажи это место в коде на которое он ругается, может можно там типчик поправить?

@ArtemMakhaydinov
Copy link
Contributor

@SlumberyDude
@nestjs/graphql без ts-morph или @nestjs/core 10.2+ версии?

@SlumberyDude
Copy link
Contributor

@ArtemMakhaydinov Если ts-morph нужен то можно его поставить в dev. Я про nestjs/core, что там за тип валится?

@ArtemMakhaydinov
Copy link
Contributor

@SlumberyDude
Если апнуть @nestjs/core, ide при билде выдает это и подчеркивает константу policy в указанном файле.

➤ YN0000: ┌ Library Build
➤ YN0000: │ packages/keto/src/guards/keto-access-control.guard.ts
➤ YN0000: │
➤ YN0000: │ Error: Object is of type 'unknown'.
➤ YN0000: │
➤ YN0000: │
➤ YN0000: │ packages/keto/src/guards/keto-access-control.guard.ts
➤ YN0000: │
➤ YN0000: │ Error: Object is of type 'unknown'.
➤ YN0000: │
➤ YN0000: │
➤ YN0000: │ packages/keto/src/guards/keto-access-control.guard.ts
➤ YN0000: │
➤ YN0000: │ Error: Object is of type 'unknown'.

@SlumberyDude
Copy link
Contributor

@ArtemMakhaydinov там у policy поменялся тип на unknown? Можешь предложить исправление?

@ArtemMakhaydinov
Copy link
Contributor

ArtemMakhaydinov commented Sep 18, 2023

@SlumberyDude при объявлении policy в вызове метода объявлен дженерик , я не силен в TS, но мне казалось, что при вызовах функций/методов, тайп не нужно указывать. Без этого все нормально билдится, чекается и тестится на последней версии @nestjs/core. Вот эта строка(в изначальном виде):

const policy = this.reflector.get<any>(ACCESS_POLICY_METADATA, context.getHandler())

@SlumberyDude
Copy link
Contributor

@ArtemMakhaydinov Вроде приемлемо. Ты кстати тег details закрываешь неправильно, ставь слеш перед details, а то у тебя иногда вложенные списки получаются)

@TorinAsakura
Copy link
Member

#281 (review)

проблема не решена

@SlumberyDude
Copy link
Contributor

проблема не решена

Какая? Вроде там все версии обновлены

packages/keto/package.json Outdated Show resolved Hide resolved
@TorinAsakura TorinAsakura requested review from Nelfimov and removed request for TorinAsakura January 8, 2024 23:22
Copy link
Member

@Nelfimov Nelfimov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

доработать API

@TorinAsakura TorinAsakura self-requested a review January 11, 2024 22:14
@Nelfimov Nelfimov force-pushed the feat/bump-keto-dependencies-versions branch from e1fb269 to d04a208 Compare January 12, 2024 12:06
@Nelfimov Nelfimov requested review from TorinAsakura and removed request for TorinAsakura January 12, 2024 14:26
@TorinAsakura TorinAsakura merged commit 72b4280 into master Jan 19, 2024
6 checks passed
@TorinAsakura TorinAsakura deleted the feat/bump-keto-dependencies-versions branch January 19, 2024 14:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Обновление зависимостей keto
5 participants