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

Chore/update button peerDependencies #485

Merged
merged 2 commits into from
Aug 25, 2023

Conversation

MakhovRoman
Copy link
Contributor

  • Апгрейд peerDependencies в ui-parts/button и ui-proto/button

@MakhovRoman MakhovRoman self-assigned this Aug 23, 2023
@MakhovRoman MakhovRoman linked an issue Aug 23, 2023 that may be closed by this pull request
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.

  • ошибка в чеках последнего коммита
  • коммит не подписан

@MakhovRoman
Copy link
Contributor Author

  • ошибка в чеках последнего коммита
  • коммит не подписан
  • Я пытался нагуглить описание ошибки, но из самого подходящего ответа понял, что проблема либо с файлами node.js, либо что-то не так с пакетами.
    Для меня непонятной остается причина возникновения ошибки - я пробовал откатить изменения, но ошибка сохраняется.

  • Я не совсем понял, что имелось ввиду - коммит я добавлял командой git commit -S -m "chore: update peerDependencies" , gpg ключ подтянулся. Или нужно было по-другом коммитить?

@Nelfimov
Copy link
Member

  • ошибка в чеках последнего коммита
  • коммит не подписан
  • Я пытался нагуглить описание ошибки, но из самого подходящего ответа понял, что проблема либо с файлами node.js, либо что-то не так с пакетами.
    Для меня непонятной остается причина возникновения ошибки - я пробовал откатить изменения, но ошибка сохраняется.

yarn check пробовал?

Так же попробуй билдить библиотеки, либо воспользуйся командой yarn workspaces changed foreach image pack --publish --tag-policy hash-timestamp --registry some. Там всплывет ошибка с подробным описанием.

  • Я не совсем понял, что имелось ввиду - коммит я добавлял командой git commit -S -m "chore: update peerDependencies" , gpg ключ подтянулся. Или нужно было по-другом коммитить?

Мой косяк, не увидел. Считай это выполнено.

@TorinAsakura
Copy link
Member

@MakhovRoman -S не нужно вводить каждый раз, в документации гитхаба есть следующая страница где описан процесс настройки автоподписи коммита

@MakhovRoman
Copy link
Contributor Author

MakhovRoman commented Aug 24, 2023

  • ошибка в чеках последнего коммита
  • коммит не подписан
  • Я пытался нагуглить описание ошибки, но из самого подходящего ответа понял, что проблема либо с файлами node.js, либо что-то не так с пакетами.
    Для меня непонятной остается причина возникновения ошибки - я пробовал откатить изменения, но ошибка сохраняется.

yarn check пробовал?

Так же попробуй билдить библиотеки, либо воспользуйся командой yarn workspaces changed foreach image pack --publish --tag-policy hash-timestamp --registry some. Там всплывет ошибка с подробным описанием.

  • Я не совсем понял, что имелось ввиду - коммит я добавлял командой git commit -S -m "chore: update peerDependencies" , gpg ключ подтянулся. Или нужно было по-другом коммитить?

Мой косяк, не увидел. Считай это выполнено.

Результат выполнения yarn check:

yarn check logs
(node:71409) [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 5s 210ms
➤ YN0000: Done in 5s 213ms
➤ YN0000: ┌ Typecheck
➤ YN0000: └ Completed in 19s 117ms
➤ YN0000: Done in 19s 117ms
➤ YN0000: ┌ Lint
➤ YN0000: └ Completed in 30s 591ms
➤ YN0000: Done in 30s 591ms

Результат выполнения yarn workspaces changed foreach image pack --publish --tag-policy hash-timestamp --registry some:

yarn workspaces logs
(node:71902) [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)
[@atls-ui-admin/icon]: Process started
[@atls-ui-admin/icon]: ➤ YN0000: Workspace @atls-ui-admin/icon not allowed for package.
[@atls-ui-admin/icon]: ➤ YN0000: Done in 0s 1ms
[@atls-ui-admin/icon]: Process exited (exit code 0), completed in 0s 270ms

[@atls-ui-admin/icons]: Process started
[@atls-ui-admin/icons]: ➤ YN0000: Workspace @atls-ui-admin/icons not allowed for package.
[@atls-ui-admin/icons]: ➤ YN0000: Done in 0s
[@atls-ui-admin/icons]: Process exited (exit code 0), completed in 0s 220ms

[@atls-ui-admin/upload]: Process started
[@atls-ui-admin/upload]: ➤ YN0000: Workspace @atls-ui-admin/upload not allowed for package.
[@atls-ui-admin/upload]: ➤ YN0000: Done in 0s
[@atls-ui-admin/upload]: Process exited (exit code 0), completed in 0s 206ms

[@atls-ui-parts/card]: Process started
[@atls-ui-parts/card]: ➤ YN0000: Workspace @atls-ui-parts/card not allowed for package.
[@atls-ui-parts/card]: ➤ YN0000: Done in 0s 1ms
[@atls-ui-parts/card]: Process exited (exit code 0), completed in 0s 224ms

[@atls-ui-proto/carousel]: Process started
[@atls-ui-proto/carousel]: ➤ YN0000: Workspace @atls-ui-proto/carousel not allowed for package.
[@atls-ui-proto/carousel]: ➤ YN0000: Done in 0s
[@atls-ui-proto/carousel]: Process exited (exit code 0), completed in 0s 224ms

Все команды выполнял в корневом каталоге - hyperion. Если я все правильно понял, то запуск в корне команды yarn install подтягивает по цепочке установку зависимостей во всех workspace ?

Пытался связать ошибку с DeprecationWarning, но найденный способ не помог. Не знаю куда смотреть, чтобы решить эту проблему. По-идее, даже если у меня криво встали зависимости, то на проверки в workflows это никак не должно было повлиять?

@TorinAsakura
Copy link
Member

  • ошибка в чеках последнего коммита
  • коммит не подписан
  • Я пытался нагуглить описание ошибки, но из самого подходящего ответа понял, что проблема либо с файлами node.js, либо что-то не так с пакетами.
    Для меня непонятной остается причина возникновения ошибки - я пробовал откатить изменения, но ошибка сохраняется.

yarn check пробовал?
Так же попробуй билдить библиотеки, либо воспользуйся командой yarn workspaces changed foreach image pack --publish --tag-policy hash-timestamp --registry some. Там всплывет ошибка с подробным описанием.

  • Я не совсем понял, что имелось ввиду - коммит я добавлял командой git commit -S -m "chore: update peerDependencies" , gpg ключ подтянулся. Или нужно было по-другом коммитить?

Мой косяк, не увидел. Считай это выполнено.

Результат выполнения yarn check:

yarn check logs
Результат выполнения yarn workspaces changed foreach image pack --publish --tag-policy hash-timestamp --registry some:
yarn workspaces logs
Все команды выполнял в корневом каталоге - hyperion. Если я все правильно понял, то запуск в корне команды yarn install подтягивает по цепочке установку зависимостей во всех workspace ?

Пытался связать ошибку с DeprecationWarning, но найденный способ не помог. Не знаю куда смотреть, чтобы решить эту проблему. По-идее, даже если у меня криво встали зависимости, то на проверки в workflows это никак не должно было повлиять?

  1. Если тебе нужно ре-ревью - перезапроси его нажав на соответствующую кнопку в сайдбаре
  2. Warning - не ошибка, она ничего не ломает, а лишь предупреждает, на неё не стоит обращать внимания
  3. yarn выполненный в корне запускает установку всех зависимостей по цепочке, да
  4. https://github.com/atls/hyperion/actions/runs/5951390633/job/16141112732?pr=485#step:4:151 - ты не закоммитил изменения своего yarn.lock

@MakhovRoman
Copy link
Contributor Author

  • ошибка в чеках последнего коммита
  • коммит не подписан
  • Я пытался нагуглить описание ошибки, но из самого подходящего ответа понял, что проблема либо с файлами node.js, либо что-то не так с пакетами.
    Для меня непонятной остается причина возникновения ошибки - я пробовал откатить изменения, но ошибка сохраняется.

yarn check пробовал?
Так же попробуй билдить библиотеки, либо воспользуйся командой yarn workspaces changed foreach image pack --publish --tag-policy hash-timestamp --registry some. Там всплывет ошибка с подробным описанием.

  • Я не совсем понял, что имелось ввиду - коммит я добавлял командой git commit -S -m "chore: update peerDependencies" , gpg ключ подтянулся. Или нужно было по-другом коммитить?

Мой косяк, не увидел. Считай это выполнено.

Результат выполнения yarn check:
yarn check logs
Результат выполнения yarn workspaces changed foreach image pack --publish --tag-policy hash-timestamp --registry some:
yarn workspaces logs
Все команды выполнял в корневом каталоге - hyperion. Если я все правильно понял, то запуск в корне команды yarn install подтягивает по цепочке установку зависимостей во всех workspace ?
Пытался связать ошибку с DeprecationWarning, но найденный способ не помог. Не знаю куда смотреть, чтобы решить эту проблему. По-идее, даже если у меня криво встали зависимости, то на проверки в workflows это никак не должно было повлиять?

  1. Если тебе нужно ре-ревью - перезапроси его нажав на соответствующую кнопку в сайдбаре
  2. Warning - не ошибка, она ничего не ломает, а лишь предупреждает, на неё не стоит обращать внимания
  3. yarn выполненный в корне запускает установку всех зависимостей по цепочке, да
  4. https://github.com/atls/hyperion/actions/runs/5951390633/job/16141112732?pr=485#step:4:151 - ты не закоммитил изменения своего yarn.lock

Спасибо, все сработало

@TorinAsakura TorinAsakura merged commit cb9e5f9 into master Aug 25, 2023
6 checks passed
@TorinAsakura TorinAsakura deleted the chore/button-peerdependencies branch August 25, 2023 16:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Грейд версий зависимостей button
3 participants