seoassistant / chrome-extension

Extensão chrome para facilitar homologação de SEO.
MIT License
6 stars 4 forks source link

Não é possível buildar o projeto. Erro ...JSON.parse #4

Closed teles closed 6 years ago

teles commented 6 years ago

Ao tentar rodar npm run build em uma máquina diferente recebo o seguinte erro:

captura de tela 2018-03-12 as 11 20 36

Provavelmente existe uma versão mínima do node para o projeto. Se for o caso é necessário deixar isso explícito no README.md.

@taciogt é esse erro que acontece com você?

taciogt commented 6 years ago

Esse mesmo erro, @teles

teles commented 6 years ago

Vou investigar. Acho que é só por causa da versão do node.

O problema já era reportado aqui no boilerplate que usei https://github.com/samuelsimoes/chrome-extension-webpack-boilerplate/issues/28

teles commented 6 years ago

Basta atualizar a versão do node para 8.qualquer-coisa. Só não sei se essa versão tem compatilidade com as versões anteriores do node.

dann95 commented 6 years ago

image image

dann95 commented 6 years ago

@teles Estranho, o spread operator diz ser compativel com a versão http://node.green/#ES2015-syntax-spread-------operator

teles commented 6 years ago

Também achei isso estranho @dann95, spread operator era pra ter compatibilidade faz tempo.

dann95 commented 6 years ago

@teles Ao parecer o spread operator só funciona com Arrays nas versões anteriores do node, pra contornar esse erro, daria pra somar os dois objetos, pois o que ele está fazendo ali é somar dois objetos (como mostra a foto) image

A solução trivial pode ser com Object.assign() https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Object/assign#Browser_compatibility

dann95 commented 6 years ago

@teles Sua intenção é dar suporte a maior quantidade de versões do Node possíveis?

teles commented 6 years ago

@dann95 sim, prefiro dar mais suporte se possível.

dann95 commented 6 years ago

Sinto cheiro de pull request saindo hihih

teles commented 6 years ago

Beleza @dann95

teles commented 6 years ago

Oi, @taciogt. Pode fazer um pull e ver se consegue rodar o projeto? Eu consegui aqui e se você também conseguir damos a issue por fechada.

dann95 commented 6 years ago

@teles pra evitar que isso acidentalmente aconteça, https://travis-ci.org/

teles commented 6 years ago

O problema está resolvido. Obrigado @dann95

Vou adicionar o travis como parte do roadmap. :D

dann95 commented 6 years ago

Se quiser, posso fazer o pr, existe algum slack ou whatsapp para debater sobre esse projeto?