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

Обновление зависимостей @grpc-playground/editor-fragment #15

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

Comments

@SlumberyDude
Copy link

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

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

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

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

  • 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>

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

shifuwww commented Sep 17, 2023

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

Мне нужно обновить пакет @grpc-playground/editor-fragment ?

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

Я должен просто написать команды которые надо использовать? Так как не указан репозиторий где нужно менять зависимости
yarn workspace <workspace> add @grpc-playground/editor-fragment@latest
yarn install
yarn check
yarn test unit

@SlumberyDude
Copy link
Author

@shifuwww Привет!

Тебе нужно обновить зависимости вот в этом репозитории.

Мне нужно обновить пакет @grpc-playground/editor-fragment ?

Да, это так

Я должен просто написать команды которые надо использовать? Так как не указан репозиторий где нужно менять зависимости

Нет, нужно добиться обновления зависимостей. Для самого обновления рекомендую использовать yarn workspace <workspace> add ...

Команды yarn check, yarn test unit нужно чтобы проверить что все работает. Начни с того, что у тебя в репозитории еще до обновления все чеки проходят.

yarn workspace add @grpc-playground/editor-fragment@latest

Это неправильно. @grpc-playground/editor-fragment это название твоего воркспейса, а пакеты, которые требуют обновления находятся в package.json.

@shifuwww
Copy link
Contributor

@SlumberyDude Понял) Спасибо

@shifuwww
Copy link
Contributor

shifuwww commented Sep 18, 2023

Привет @SlumberyDude

Тебе нужно обновить зависимости вот в этом репозитории.

"react-ace": "patch:react-ace@npm%3A9.4.0#~/.yarn/patches/react-ace-npm-9.4.0-c0e2dac510.patch"
Не понял как обновить эту зависимость, при обновлении через:
yarn workspace @grpc-playground/editor-fragment add react-ace@latest ставится версия 10.1.0 но без пути, из за этого не проходит typecheck (пробовал ставить 9.4.0 версию)

@SlumberyDude
Copy link
Author

@shifuwww Привет. Тут у react-ace стоит пропатченная версия, думаю патч был вызван как раз тем, что не проходили тайпчеки. Необходимо установить более позднюю версию и пропатчить ее аналогичным образом, чтобы проходили тайпчеки. Про патчи в ярне можешь почитать тут

@shifuwww
Copy link
Contributor

@SlumberyDude Есть ли правила именования веток? Можно ли назвать ветку:
git branch enhancement:@grpc-playground/editor-fragment?

@SlumberyDude
Copy link
Author

@shifuwww Есть. Ветка должна начинаться с ключевого слова из конвенции. Доступные слова : feat, fix, chore, test , затем слеш и произвольное название. Например, feat/update-dependency

@shifuwww
Copy link
Contributor

@SlumberyDude
Copy link
Author

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

Делай PR этой ветки в мастер. Ставь assign себя в этот PR и запрашивай у меня ревью, когда посчитаешь, что с таской все.

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

2 participants