Обновление зависимостей `dataloader`

SlumberyDude commented 1 year ago

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

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

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

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

Исключения:

Зависимости 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 commented 1 year ago

Пакет 'rxjs' в devDependencies.

Обновляю до последней версии с помощью 'yarn workspace @atls/nestjs-dataloader add rxjs@7.8.1 -D'. В package.json версия обновляется, но вебшторм подчеркивает версию, при наведении пишет 'The installed version "rxjs@7.5.4" doesn't match the version range "7.8.1" '.

Yarn install не обновляет версию rxjs.

Как обновить что бы потестить?

SlumberyDude commented 1 year ago

@ArtemMakhaydinov Я когда-то делал заплатку на версию rxjs в пакете dataloader. Эта заплатка в корневом package.json

"resolutions": {
    "@atls/nestjs-dataloader/rxjs": "7.5.4",

Ее теперь можно убрать.

ArtemMakhaydinov commented 1 year ago

Все зависимости успешно обновлены до последних версий.

Следующие команды ошибок не вызывают: yarn install yarn workspace @atls/nestjs-dataloader run build yarn check yarn test unit

Бренч feat/dataloader-deps-update Коммит feat: bump dataloader dependecies version

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

TorinAsakura commented 1 year ago

@ArtemMakhaydinov Полагаю, нужно всё-таки прочитать конвенцию нормально… Как минимум в части работы с гитом у тебя не должно возникать вопросов, это всё равно что уметь писать - базовый навык.

SlumberyDude commented 1 year ago

@ArtemMakhaydinov Вроде ок. Создавай Pull Request в мастер и потом запрашивай у меня ревью на PR

SlumberyDude commented 1 year ago

@ArtemMakhaydinov На локале чеки проходят? В ПР они валятся на линте

ArtemMakhaydinov commented 1 year ago

@SlumberyDude

artem@artem-To-be-filled-by-O-E-M:~/Dev/Atlantis/nestjs$ yarn check (node:49155) [DEP0005] DeprecationWarning: Buffer() is deprecated due to security and usability issues. Please use the Buffer.alloc(), Buffer.allocUnsafe(), or Buffer.from() methods instead. (Use `node --trace-deprecation ...` to show where the warning was created) ➤ YN0000: ┌ Format ➤ YN0000: └ Completed in 3s 990ms ➤ YN0000: Done in 3s 992ms ➤ YN0000: ┌ Typecheck ➤ YN0000: └ Completed in 21s 269ms ➤ YN0000: Done in 21s 269ms ➤ YN0000: ┌ Lint ➤ YN0000: └ Completed in 15s 971ms ➤ YN0000: Done in 15s 971ms

Чек, тест, билд - все норм.

SlumberyDude commented 1 year ago

@TorinAsakura Это ошибка чека относится к проекту? Или это что-то гитхабовское

atls / nestjs