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

M2#A51 - primordials is not defined #551

Closed
alexandre1202 opened this issue Jun 24, 2021 · 5 comments
Closed

M2#A51 - primordials is not defined #551

alexandre1202 opened this issue Jun 24, 2021 · 5 comments
Labels

Comments

@alexandre1202
Copy link

alexandre1202 commented Jun 24, 2021

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
Copy link
Member

fdaciuk commented Jun 27, 2021

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 =)
instalar as dependências que estão listadas da aula M2#A04 e M2#A07, e escrever seus testes =)

@fdaciuk fdaciuk changed the title M2#A51 - yarn add --dev snazzy M2#A51 - primordials is not defined Jun 27, 2021
@fdaciuk fdaciuk pinned this issue Jun 27, 2021
@alexandre1202
Copy link
Author

alexandre1202 commented Jul 2, 2021

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
Copy link
Member

fdaciuk commented Jul 2, 2021

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
Copy link
Author

alexandre1202 commented Jul 3, 2021 via email

@fdaciuk
Copy link
Member

fdaciuk commented Jul 3, 2021

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants