-
Notifications
You must be signed in to change notification settings - Fork 315
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
Chore: Change code samples following the Runkit pattern #140
base: master
Are you sure you want to change the base?
Conversation
4 similar comments
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 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@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') |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
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 😬
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 😬 🙏 |
Valeu @lucianopf, rebaseei e fiz os ajustes! ✌️ |
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 😬 |
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? 🙏
Assim que terminar o push como o CI foi "corrigido" acho que vai passar e podemos seguir com o merge 🙏 |
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 apenascep
como mostrado na imagem abaixo:Runkit
Readme