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

Обновление зависимостей @atls/grpc-playground-app #20

Closed
SlumberyDude opened this issue Sep 17, 2023 · 7 comments
Closed
Assignees
Labels
enhancement New feature or request

Comments

@SlumberyDude
Copy link

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

Обновить версии пакетов в зависимостях

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

Обновить версии зависимости до максимально возможных и зафиксировать таким образом, чтобы ничего не ломалось. Должны без ошибок проходить команды:

  • yarn install
  • yarn check
  • yarn test unit

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

Исключения:

typescript обновлять не нужно.

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

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

@SlumberyDude SlumberyDude added the enhancement New feature or request label Sep 17, 2023
@shifuwww
Copy link
Contributor

@SlumberyDude Привет, это последний мой таск, нужно ли мне тегать после Андрея?

@SlumberyDude
Copy link
Author

@shifuwww Привет, я у него запрашиваю ревью после того как сам посмотрю обычно. Если все в этой репе вычистим то посмотрим, думаю пока что не нужно

@shifuwww
Copy link
Contributor

shifuwww commented Sep 21, 2023

@SlumberyDude Привет, не могу найти решение ошибки при typecheck после установки пакета "next": "13.4.9", в .yarn/__virtual__/next-virtual-b7c3883125/5/home/shifuwww/.yarn/berry/cache/next-npm-13.4.9-5c37513773-9.zip/node_modules/next/dist/server/base-http/node.d.ts
ошибка : Error: Type 'ServerResponse' is not generic.

Делал поиск по проекту самое близкое что нашел это export interface ServerReflectionResponse, так же пробовал ставить разные версии, при любой отличной от "next": "12.3.4", ошибка

Можно узнать где про это почитать?

@SlumberyDude
Copy link
Author

@shifuwww Привет, если хочешь найти конкретную причину ошибки, то я обычно смотрю в каком файле исходной либы (в данном случае next) возникает ошибка, нахожу его на гитхабе, смотрю его гит историю и иногда нахожу конкретный коммит, который меняет тип например. У некста вроде с 13.4 большие ломающие изменения пошли.

Вот тот файл, но дженериков я там не вижу
https://github.com/vercel/next.js/blob/canary/packages/next/src/server/base-http/node.ts

@shifuwww
Copy link
Contributor

@SlumberyDude Ооо круто) Понял, спасибо

@Nelfimov
Copy link
Member

Некст в наших проектах используем не выше 12.3.4

@SlumberyDude
Copy link
Author

Закрыто в связи с мержем пулл реквеста

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants