search

BrasilAPI / cep-promise

Busca por CEP integrado diretamente aos serviços dos Correios, ViaCEP e outros (Node.js e Browser)
MIT License
2.88k stars 316 forks source link

Problema entre "npm run build" para o bower e .gitignore #86

Closed filipedeschamps closed 7 years ago

filipedeschamps commented 7 years ago

@jhonnymoreira tudo bem?

Fiz o merge e iria fazer o release, mas ao rodar local o npm run build notei duas coisas:

  1. Gerou um diff dos binários finais de distribuição (não sei porque)
  2. Listou arquivos untracked de dist. Pelo que eu entendo, eles deveriam estar no .gitignore, correto? Ou precisam estar commited?

Segue print do git status logo após o npm run build da branch master limpa:

image

lucianopf commented 7 years ago

Então.. pelo que andei futucando no bower eu tbm não consegui engatilhar o npm run build.

Daí acredito que o @jhonnymoreira achou que a melhor solução seria versionar ao menos os arquivos do browser como é feito por exemplo no Bootstrap.

Eu acho uma falha terrível no Bower mas acredito que esse é o jeito padrão de se publicar pacotes no registry deles. =/

Acho que da pra fazer um ignore melhorzinho tipo:

dist
!dist/cep-promise-browser[!.min]js

(Nunca usei essa opção da lista não, mas acho que funciona! hahaha)

lucianopf commented 7 years ago

A vantagem de fazer dessa forma é que pode-se acessar esses arquivos por CDN.

jhonnymoreira commented 7 years ago

@lucianopf @filipedeschamps já criei um registro no bower/registry. To esperando mais um tempinho pra ver qq tá rolando, pq a pasta dist/ não tá sendo reconhecida. [=

filipedeschamps commented 7 years ago

Show!

Acho que do .gitignore é o que o @lucianopf falou mesmo 👍

lucianopf commented 7 years ago

Fiz o PR #87 pra minimizar esses efeitos =D

(O yarn é pq ele realmente ta bem melhor que o atual NPM 😞 )

filipedeschamps commented 7 years ago

Show, tudo ajustado!

Fiz release da versão 2.0.5, belo trabalho pessoal 👍