Skip to content

⬆️ Upgrade major dependencies#72

Open
leonardobrito wants to merge 2 commits into
upgrade-minor-depsfrom
upgrade-major-deps
Open

⬆️ Upgrade major dependencies#72
leonardobrito wants to merge 2 commits into
upgrade-minor-depsfrom
upgrade-major-deps

Conversation

@leonardobrito

Copy link
Copy Markdown
  • O que esta PR faz / Porque nós precisamos fazer isso?
    Upgrade major deps.

  • Que tipo de mudança foi introduzida?

  • Bugfix 🐛
  • Feature ✨
  • Refactor 🔨
  • Build-Mudança relatada 🔖
  • Outro, por favor descrever:
  • Que tipo de teste foi implementado? ✅
  • Unitário
  • Integração/Headless
  • Nenhum, porque:

Em caso de dúvidas consultar a documentação.

@leonardobrito
leonardobrito requested review from vhoyer and vitebo March 10, 2022 17:59

@vhoyer vhoyer left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

caragem atualizar tudo assim haha, ta tudo certinho ae?

@leonardobrito

leonardobrito commented Mar 10, 2022

Copy link
Copy Markdown
Author

caragem atualizar tudo assim haha, ta tudo certinho ae?

Aparentemente não @vhoyer tá dando conflitos com os pkgs internos:

npm i                                                                            (arn:aws:eks:us-east-1:725582217686:cluster/homolog-cluster)
npm ERR! code ERESOLVE
npm ERR! ERESOLVE could not resolve
npm ERR! 
npm ERR! While resolving: @quero/eslint-config-react@1.1.0
npm ERR! Found: eslint-plugin-jest-dom@4.0.1
npm ERR! node_modules/eslint-plugin-jest-dom
npm ERR!   dev eslint-plugin-jest-dom@"^4.0.1" from the root project
npm ERR! 
npm ERR! Could not resolve dependency:
npm ERR! peer eslint-plugin-jest-dom@"^3.9.0" from @quero/eslint-config-react@1.1.0
npm ERR! packages/eslint-config-react
npm ERR!   @quero/eslint-config-react@1.1.0
npm ERR!   node_modules/@quero/eslint-config-react
npm ERR!     workspace packages/eslint-config-react from the root project
npm ERR! 
npm ERR! Conflicting peer dependency: eslint-plugin-jest-dom@3.9.4
npm ERR! node_modules/eslint-plugin-jest-dom
npm ERR!   peer eslint-plugin-jest-dom@"^3.9.0" from @quero/eslint-config-react@1.1.0
npm ERR!   packages/eslint-config-react
npm ERR!     @quero/eslint-config-react@1.1.0
npm ERR!     node_modules/@quero/eslint-config-react
npm ERR!       workspace packages/eslint-config-react from the root project

Acho que essa eu vou fechar e fazer com mais calma. 😭

Ele só não quebra os teste e nem nada, ia ter ver com vocês para saber como valida se tá certo mesmo. cc.: @vitebo @vhoyer

@vhoyer

vhoyer commented Mar 10, 2022

Copy link
Copy Markdown
Contributor

F

Históricamente, o que eu tenho feito pra testar essas coisinha ai, é abrir um projeto do lado dele, usando o npm link pra ver se ta tudo funcionando manualmente mesmo kkkk, meio... não escalavel, mas enfim 🤷

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.

3 participants