Skip to content
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

Website #141

Open
wants to merge 57 commits into
base: master
Choose a base branch
from
Open

Website #141

wants to merge 57 commits into from

Conversation

rafaelcamargo
Copy link

@rafaelcamargo rafaelcamargo commented Mar 10, 2019

Ref.: https://github.com/filipedeschamps/cep-promise/issues/97

Olá Pessoal!

Desculpe a demora na implementação do site. Finalmente, a concluí conforme sugestão que havia feito a vocês há pouco mais de um mês. Dito isto, existem três questões em aberto:

  1. Como vocês podem notar, foi escrita uma quantia considerável de código. Vocês preferem manter o código do site no mesmo repositório ou preferem criar um outro repositório exclusivo para o site?
  2. Onde vocês gostariam de publicar o site? Na issue, falou-se sobre Github Pages. A princípio, não escrevi nada em relação ao deploy do site. Eu particularmente tenho usado o Firebase Hosting para os sites estáticos que tenho criado para os meus projeto e tem sido uma boa experiência.
  3. No CI, embora eu não tenho tocado no código da lib, existe um teste falhando. Detalhes abaixo:
when invoked with an inexistent "99999999" CEP
      1) should reject with "service_error"

Preview

2019-03-10 19 24 32

Por ora, é isto. Aguardo as considerações de vocês 👍

@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling 6f47f4e on rafaelcamargo:website into 69aa725 on filipedeschamps:master.

3 similar comments
@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling 6f47f4e on rafaelcamargo:website into 69aa725 on filipedeschamps:master.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling 6f47f4e on rafaelcamargo:website into 69aa725 on filipedeschamps:master.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling 6f47f4e on rafaelcamargo:website into 69aa725 on filipedeschamps:master.

@lucianopf
Copy link
Member

Cara isso é ANIMAL!! ❤️

Eu pretendo revisar isso direitinho pra gente soltar uma obra prima, mas no momento ando realmente sem tempo pra dar uma olhadinha no projeto =(

Mas de qualquer forma já merece um 10/10 só pela iniciativa! 👏 👏 👏

cc: @filipedeschamps

@rafaelcamargo
Copy link
Author

@lucianopf Tranquilo cara, façam as coisas no tempo de vocês. Pra mim, tá de boas total 😉

No mais, fico feliz que tenham curtido o resultado 🎉

rfoel
rfoel previously approved these changes Aug 29, 2019
@mizamelo
Copy link

mizamelo commented May 4, 2020

Galera, o que houve com essa PR? Muito top essa implementação!

@SkyaTura
Copy link

SkyaTura commented May 6, 2020

Acho que o projeto tá meio parado, eu chuto que todos estejam enrolados com a situação da pandemia...

@lucianopf
Copy link
Member

É isso mesmo, o tendo que trabalhar 2x mais durante essa crise e ta complicado dar atenção devida aos projetos Open 😢

Já ajuda bastante os reviews da comunidade!! ^^

(Esse PR em específico eu to esperando o parecer do @filipedeschamps )

https://github.com/filipedeschamps/cep-promise/pull/141#issuecomment-472457293

@williamtorres1
Copy link

Ref.: #97

Olá Pessoal!

Desculpe a demora na implementação do site. Finalmente, a concluí conforme sugestão que havia feito a vocês há pouco mais de um mês. Dito isto, existem três questões em aberto:

  1. Como vocês podem notar, foi escrita uma quantia considerável de código. Vocês preferem manter o código do site no mesmo repositório ou preferem criar um outro repositório exclusivo para o site?

Eu, particularmente, acho melhor um repositório separado somente para o site da documentação.

@lucianopf
Copy link
Member

Minha nossa, to dando uma geral nos repos da org BrasilAPI e me deparei com esse que ta aqui esquecido há mais de ano 😱 😭

Primeiramente perdão pela ausência de atividade nesse PR!

@rafaelcamargo dado que agora o projeto foi movido para a org BrasilAPI que já possui um site, será que não faz sentido pensar numa forma de integrar isso ou seguir com a ideia do @williamtorres1?

@rafaelcamargo
Copy link
Author

@lucianopf Olá meu caro! Acho que a ideia de deixar o site num repo separado faz todo sentido sim. Se vocês puderem, criem esse repo na org que eu movo essa implementação pra ele. Valeu!

@williamtorres1
Copy link

Alguma novidade do andamento dessa PR?

@LorhanSohaky LorhanSohaky dismissed stale reviews from ghost and rfoel via 51e8815 January 22, 2023 13:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants