-
Notifications
You must be signed in to change notification settings - Fork 0
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
Обновление зависимостей cqrs-adapter
и buckets-config-adapter
#22
Comments
@shifuwww Только называй ветки лаконичнее чем в прошлом проекте, там были слишком длинные. Названия должны быть такими, чтобы мы не пустали таски, они должны отражать суть таски и при этом желательно иметь более компактное название. |
@SlumberyDude привет, после обновления пакета |
@shifuwww Ты обновил зависимость только в модуле |
@SlumberyDude Привет
нет в buckets-config-adapter |
@shifuwww Хм, понял. Понять бы, почему так происходит и как можно пофиксить. Потому что судя по коду в file.repository.ts все норм, непонятно, почему он перестает воспринимать объект File как AggregateRoot, хотя вроде File экстендитAggregateRoot в domain-module. |
Может из за расхождений версий? В domain-module еще старая версия |
@shifuwww Да, понятно, что обновление версии в модуле |
Пробовал сделать обнолвение везде, ошибка с типами не уходит. Пробовал искать версию, но если
|
@shifuwww Обновить только в infrastructure модуле недостаточно, чтобы ошибка ушла нужно обновление в entrypoint и в infrastructure. Но так как мы до них еще не дошли, то сделай эту заплатку через корневой package.json |
@shifuwww Как дела с таской? Спрашивай если застрял |
Обновил |
@shifuwww Смотри в сторону |
@SlumberyDude Привет) Тут у меня один тест не прошел: Login to Google Container Registry, ошибка: Error: Cannot perform an interactive login from a non TTY device. Это мне докер нужно настроить с авс или это что то другое? |
@shifuwww Привет, какой тест? Тут похоже на ошибку связанную с докером, поэтому если это интеграционные тесты не проходят, то думаю из-за него. Ты использовал докер раньше? Эти тесты проходят на ветке мастер без внесенных изменений или тоже с такой ошибкой ложатся? |
@SlumberyDude Привет) Тест первый: Preview / Preview (pull_request). Остальные прошли успешно. Если ты про тесты yarn test unit? То все проходит нормально. Докер раншье использовал, но только для деплоя. Тест упал во время пр, когда гит запускает тесты |
@shifuwww Да, вижу. Думаю тут проблема с ci гитхаба и настройкой workflow. |
С чем связан запрос на фичу?
Обновить версии пакетов в зависимостях
Расскажите как вы это себе видите
Обновить версии зависимости до максимально возможных и зафиксировать таким образом, чтобы ничего не ломалось. Должны без ошибок проходить команды:
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>
The text was updated successfully, but these errors were encountered: