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

Conversão para TS #169

Open
marceloF5 opened this issue May 28, 2020 · 10 comments
Open

Conversão para TS #169

marceloF5 opened this issue May 28, 2020 · 10 comments

Comments

@marceloF5
Copy link

marceloF5 commented May 28, 2020

Pessoal,

O que vocês acham de converter todo o projeto para TS? Já falaram sobre isso? Posso ajudar!

@marceloF5 marceloF5 changed the title Conversão do projeto para TS Conversão para TS May 28, 2020
@bearkfear
Copy link
Contributor

Eu posso ajudar, estava buscando exatamente isso. Se era feito em TypeScript. Os benefícios são óbvios e quem mais sai ganhando são os devs e contribuidores com a "auto-documentação".

@DavidNery
Copy link

Sim. Tanto o cep-promise quanto o BrasilAPI.
Seria ótimo.

@GustavoMilitao
Copy link

Tenho interesse em contribuir também, se ainda não começaram

@bearkfear
Copy link
Contributor

Galera, se for de interesse do adm, posso criar uma PR convertendo tudo para Typescript. Mas teria que ver qual sistema de build usar...

@felinto-dev
Copy link

Galera, se for de interesse do adm, posso criar uma PR convertendo tudo para Typescript. Mas teria que ver qual sistema de build usar...

Que tem demanda, certamente tem. Só verificar os últimos issues e boa parte deles fazem referência ao suporte para TypeScript.
No entanto, pode ser um pouco frustrante que os administradores não tenham conseguido dedicar tempo para revisar as PR
#222
@lucianopf

Então talvez você queira combinar com um dos administradores antes.

@fontebasso
Copy link
Contributor

fontebasso commented Jul 22, 2021

Além de concordar com o que @felinto-dev citou acima, gostaria de ainda tentar completar...

Tenho a impressão que esta biblioteca atingiu seu objetivo e deve receber atualizações de segurança para se manter ativa. Uma reviravolta para reescrita sob base em TS deveria ser um fork, mas para manter a galera focada e crescer junto, sinto que é uma iniciativa que os adms da org BrasilAPI devem liderar.

Claro que poderíamos subir uma nova versão com break changes a 5.0.0, mas acredito que até mesmo o nome cep-promise não está de acordo com o que percebo ser a intenção de quem se envolve com issues e PR's, que na minha opinião está mais para algo como: "O jeito definitivo de consulta de cep no Brasil com JavaScript".

@felinto-dev
Copy link

Além de concordar com o que @felinto-dev citou acima, gostaria de ainda tentar completar...

Tenho a impressão que esta biblioteca atingiu seu objetivo e deve receber atualizações de segurança para se manter ativa. Uma reviravolta para reescrita sob base em TS deveria ser um fork, mas para manter a galera focada e crescer junto, sinto que é uma iniciativa que os adms da org BrasilAPI devem liderar.

Claro que poderíamos subir uma nova versão com break changes a 5.0.0, mas acredito que até mesmo o nome cep-promise não está de acordo com o que percebo ser a intenção de quem se envolve com issues e PR's, que na minha opinião está mais para algo como: "O jeito definitivo de consulta de cep no Brasil com JavaScript".

Fonte, entendi o seu ponto a respeito de que corrigir o suporte 100% para o TypeScript seria um desafio pois muita das pessoas que mantem o projeto atualmente (creio eu) não tem conhecimento avançado em TypeScript. O Filipe Deschamps, fundador do projeto e idealizador da biblioteca, postou um vídeo a 10 meses atrás (contando o momento que estou escrevendo isso) dizendo que iria começar a aprender TypeScript (https://www.youtube.com/watch?v=M57zuClvRbM), sendo que existem issues do projeto desde 2016, o que significa que a biblioteca não foi criada com TypeScript em mente.

No entanto, vale lembrar que a biblioteca já tem um suporte, mesmo que baixo ao TypeScript, então não seria um fork e sim uma melhoria no recurso existente.

@bearkfear
Copy link
Contributor

Galera, reescrever a lib em typescript não é nada custoso, mas, que precisa de atenção dos ADMs, porque é um mega PR, visto que todo o código será reescrito. Dai, eu acredito que está iniciativa deve estar unida entre todos os que acham necessário e com o Adm é claro!

@bearkfear
Copy link
Contributor

Galera, criei uma branch, estou reescrevendo os códigos em TypeScript. Se alguém estiver a fim de apoiar => https://github.com/bearkfear/cep-promise/tree/refactor/convert-to-ts

@victorfernandesraton
Copy link

Eu pensei em reescrever do zero usando novas funcionalidades como promise.any nativo das novas versões do ecma262

https://github.com/victorfernandesraton/cep-any

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

7 participants