Chore: Change code samples following the Runkit pattern#140

Open

lnfnunes wants to merge 1 commit intoBrasilAPI:masterfrom

lnfnunes:npm-runkit

lnfnunes commented Mar 10, 2019

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 😉

coveralls commented Mar 10, 2019

Coverage remained the same at 100.0% when pulling ad22067 on lnfnunes:npm-runkit into 69aa725 on filipedeschamps:master.

4 similar comments

coveralls commented Mar 10, 2019

Coverage remained the same at 100.0% when pulling ad22067 on lnfnunes:npm-runkit into 69aa725 on filipedeschamps:master.

coveralls commented Mar 10, 2019

Coverage remained the same at 100.0% when pulling ad22067 on lnfnunes:npm-runkit into 69aa725 on filipedeschamps:master.

coveralls commented Mar 10, 2019

Coverage remained the same at 100.0% when pulling ad22067 on lnfnunes:npm-runkit into 69aa725 on filipedeschamps:master.

coveralls commented Mar 10, 2019

Coverage remained the same at 100.0% when pulling ad22067 on lnfnunes:npm-runkit into 69aa725 on filipedeschamps:master.

coveralls commented Mar 10, 2019 •

edited

Loading

Coverage remained the same at 100.0% when pulling de49344 on lnfnunes:npm-runkit into 9bca1c4 on filipedeschamps:master.

lnfnunes force-pushed the npm-runkit branch from ad22067 to af4f973 Compare

March 10, 2019 22:42

Author

lnfnunes commented Mar 10, 2019

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

lucianopf reviewed

View reviewed changes

Member

lucianopf left a comment

@lnfnunes dei uma olhdinha e curti a proposta! 🤘

Fiquei com apenas uma dúvida, qual a necessidade de alterar arquivos no dist tendo em vista que só foi alterado documentação e testes 🤔

Seria possível vc remover essas alterações do seu PR?

test/unit/cep-promise.spec.js Outdated

    
                    const cepPromise = cep('05010000')

                    expect(cepPromise.then).to.be.a('function')

                    expect(cepPromise.catch).to.be.a('function')

                    const lib = cepPromise('05010000')

Member

lucianopf Jul 31, 2020

Hehehe, nesse caso o cepPromise fazia bastante sentido, pq de fato é a promise retornada pela execução da lib hehehe, mas não vejo problema em manter como lib 😬

Member

lucianopf commented Jul 31, 2020

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 😬 🙏

lucianopf assigned lnfnunes

lucianopf added the enhancement label


          Chore: Change code samples following the Runkit pattern

de49344

lnfnunes force-pushed the npm-runkit branch from af4f973 to de49344 Compare

October 14, 2020 16:55

Author

lnfnunes commented Oct 14, 2020

Valeu @lucianopf, rebaseei e fiz os ajustes! ✌️

Member

lucianopf commented Nov 19, 2020

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 😬

Member

lucianopf commented Dec 29, 2020

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 🙏

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels