cep-promise
cep-promise copied to clipboard
BrasilAPI
Chore: Change code samples following the Runkit pattern
Alteração simples nos códigos de exemplo ("Como utilizar") de forma que se a pessoa simplesmente consiga copiar e colar no Runkit e o código funcione!
Pois o padrão do Runkit é seguindo o nome da lib e neste caso fica cepPromise e não apenas cep como mostrado na imagem abaixo:
Runkit
Readme
- PS: Alterei o import também nos testes unitários para manter um padrão 😉
Coverage remained the same at 100.0% when pulling ad2206755f496d9f72da7acbc1acbe881d1aeefd on lnfnunes:npm-runkit into 69aa7259cb6211bcb8f7ff14f88e7c90d7f91c20 on filipedeschamps:master.
Coverage remained the same at 100.0% when pulling ad2206755f496d9f72da7acbc1acbe881d1aeefd on lnfnunes:npm-runkit into 69aa7259cb6211bcb8f7ff14f88e7c90d7f91c20 on filipedeschamps:master.
Coverage remained the same at 100.0% when pulling ad2206755f496d9f72da7acbc1acbe881d1aeefd on lnfnunes:npm-runkit into 69aa7259cb6211bcb8f7ff14f88e7c90d7f91c20 on filipedeschamps:master.
Coverage remained the same at 100.0% when pulling ad2206755f496d9f72da7acbc1acbe881d1aeefd on lnfnunes:npm-runkit into 69aa7259cb6211bcb8f7ff14f88e7c90d7f91c20 on filipedeschamps:master.
Coverage remained the same at 100.0% when pulling ad2206755f496d9f72da7acbc1acbe881d1aeefd on lnfnunes:npm-runkit into 69aa7259cb6211bcb8f7ff14f88e7c90d7f91c20 on filipedeschamps:master.
Coverage remained the same at 100.0% when pulling de49344633ba02c8a3c21e2d8541c5b4c949755c on lnfnunes:npm-runkit into 9bca1c4aec436667fcee7d7c837f5c8b2b752a6f on filipedeschamps:master.
Obs: Aparentemente o e2e está quebrando um teste, porém não é devido a esta implementação e a correção estou entendendo que já esta aberta no PR #139
Obs: Acho que ta falhando por conta de um problema antigo, acho que faz sentido fazer um rebase com a master pra manter tudo atualizadinho, daí deve voltar a passar :grimacing: :pray:
Mestres, só pra não deixar vcs perdidos abri uma issue com um planinho de ação pra reorganizar o repo dado a migração pra org do BrasilAPI 😬
https://github.com/BrasilAPI/cep-promise/issues/197
Bom dia mestre @lnfnunes !
Primeiramente perdão a demora 😢
Bora seguir com esse PR? Eu tentei fazer o rebase pra vc mas a branch tava protegida e precisarei da sua ajuda 😢
Pode fazer os seguintes passos por favor? 🙏
git remote add upstream https://github.com/BrasilAPI/cep-promise.git
git fetch --all
git rebase upstream/master
git push --force-with-lease
Assim que terminar o push como o CI foi "corrigido" acho que vai passar e podemos seguir com o merge 🙏