cep-promise icon indicating copy to clipboard operation
cep-promise copied to clipboard

Conversão para TS

Open marceloF5 opened this issue 4 years ago • 10 comments

Pessoal,

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

marceloF5 avatar May 28 '20 08:05 marceloF5

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".

bearkfear avatar Nov 17 '20 03:11 bearkfear

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

DavidNery avatar Dec 02 '20 10:12 DavidNery

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

GustavoMilitao avatar Mar 03 '21 01:03 GustavoMilitao

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

bearkfear avatar Jul 21 '21 19:07 bearkfear

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 https://github.com/BrasilAPI/cep-promise/pull/222 @lucianopf

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

felinto-dev avatar Jul 22 '21 19:07 felinto-dev

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".

fontebasso avatar Jul 22 '21 23:07 fontebasso

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.

felinto-dev avatar Jul 22 '21 23:07 felinto-dev

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 avatar Jul 23 '21 00:07 bearkfear

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

bearkfear avatar Jul 23 '21 02:07 bearkfear

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

victorfernandesraton avatar Mar 04 '22 18:03 victorfernandesraton