da2k / curso-reactjs-ninja

916 stars 323 forks source link

M2#A51 - primordials is not defined #551

Closed alexandre1202 closed 3 years ago

alexandre1202 commented 3 years ago

Olá Fernando.

Tudo bom?

Eu acabei de executar o comando para adicionar o snazzy e vi que no meu caso demorou muito mais tempo e apresentou muito mais informação que no video do curso. Sei que as dependências estão em constante mudanças, atualizações e melhorias porém poderia por favor, passar o olho no arquivo em anexo e ver ser o que tinha para ser feito realmente está tudo correto?

yarn-add-snazzy-console.rtf.zip

Depois da configuração do lint:watch: image eu tentei executar o comando yarn lint:watch e eu recebo a mensagem de erro abaixo: image

Por via das dúvidas eu deixei o gulpfile.js assim: image

Subi o projeto temporariamente aqui nesse repositório

Hoje eu notei que o comando abaixo também não está funcionando: image

Estraguei tudo aqui

@fdaciuk

fdaciuk commented 3 years ago

Oi @Alexandre1202! No seu log de pós instalação do snazzy está tudo certo! Ali no início alguns warnings são exibidos, mas nada que seja um problema.. problema seria se o comando finalizasse com erro, o que não foi o caso =)

Sobre o erro do gulp, acabei de descobrir que tem uma outra configuração que precisa ser usada.

Pelos seus prints, você está usando o Yarn na v1, certo? Então só adicione essa chave resolutions em algum lugar dentro do seu package.json (pode ser lá no final do arquivo, antes do fechamento da última chave):

  "resolutions": {
    "graceful-fs": "^4.2.4"
  }

Se mais alguém encontrar essa issue pelo mesmo problema, e estiver usando o NPM, a solução está no link acima, mas é basicamente adicionar um script de preinstall no package.json, além da entrada resolutions acima. Ficaria algo assim:

{
  // Your current package.json
  "scripts": {
    // Your current package.json scripts
    "preinstall": "npx npm-force-resolutions"
  },
  "resolutions": {
    "graceful-fs": "^4.2.4"
  }
}

E sobre a parte de testes, é porque no seu projeto você não fez a instalação do Jest =) Só instalar as dependências que estão listadas da aula M2#A04 e M2#A07, e escrever seus testes =)

alexandre1202 commented 3 years ago

Oi Fernando.

Tudo funcionando a partir das suas orientações.

Aproveitando o momento que foi apresentado a utilização do Yarn ao invés do Npm, então deixa eu perguntar. Desde o inicio estamos utilzando o npm e depois mudamos para o yarn. Então me pergunto. A partir do momento que passei a usar o yarn eu poderia voltar a usar o npm novamente ou de agora em diante devo sempre seguir com yarn?

Pergunto isso porque no link que os passos para instalação do Jest ainda usa o npm mas eu já estava utilizando o yarn. Quando eu rodei o yarn lint:watch após as instalação utilizando o npm que foi indicado não funcionou aqui.

image

E após eu voltar a usar o yarn para fazer as dependencias do Jest então funcionou: image

Resumindo. Quando começar a usar o yarn então devemos seguir com ele sempre, certo?

@fdaciuk

fdaciuk commented 3 years ago

Oi @Alexandre1202! Legal que tá rodando tudo certo :D

Sobre Yarn vs NPM, o ideal é usar sempre um ou outro. Você pode usar o Yarn em um projeto, e o NPM em outro, mas nunca os dois juntos no mesmo projeto =)

Isso falando apenas da parte de instalação de dependências. Usar para rodar npm scripts não deve - ao menos não deveria - fazer diferença =)

alexandre1202 commented 3 years ago

Obrigado pela explicação parceiro

Pode fechar

On Fri, 2 Jul 2021 at 15:38 Fernando Daciuk @.***> wrote:

Oi @Alexandre1202 https://github.com/Alexandre1202! Legal que tá rodando tudo certo :D

Sobre Yarn vs NPM, o ideal é usar sempre um ou outro. Você pode usar o Yarn em um projeto, e o NPM em outro, mas nunca os dois juntos no mesmo projeto =)

Isso falando apenas da parte de instalação de dependências. Usar para rodar npm scripts não deve - ao menos não deveria - fazer diferença =)

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/da2k/curso-reactjs-ninja/issues/551#issuecomment-873187997, or unsubscribe https://github.com/notifications/unsubscribe-auth/AB5CMVJQUZQ35FWBFVLYACDTVYBS3ANCNFSM47I2WWJA .

-- ------------------------- Alexandre Barbosa 11-99169-3870 (Claro)

fdaciuk commented 3 years ago

Show! Qualquer dúvida, só avisar :D