Criação de testes Unitários e projeto baseado no poetry

[joepreludian]

Hey @lipe14-ops! Bom dia, man! Estava dando uma olhada no seu repositório e gostei bastante da qualidade do código escrito! Parabéns!

Eu gostaria de saber se voce está aberto a aceitar PRs com algumas melhorias. Das quais eu consigo me recordar agora, seriam 5:

• Adicionar Testes e Cobertura de código; Pensei em criar um projeto simples em cima do pyenv pra testar sua API.
• Criar uma ação no github actions pra validar eventuais implementações no seu codebase;
• Usar um projeto baseado no poetry. Acho ele bacana pois pode nos ajudar a manter um track das versões por meio dos lock file ao mesmo tempo que dá pra ser utilizado em instalações em modo desenvolvimento. Ele já encapsula o projeto dentro de um virtual env e conversa bem com o pyenv.
• Adicionar rotinas do pre-commit para pré-validações de código, linting, etc.
• Adicionar o __pycache__ nos arquivos do gitignore.

Acredito que, com esses pontos, seu projeto poderia ser facilmente ser mantido sem perder a qualidade e esmero que voce colocou nele.

Bom, mas é só uma sugestão, tá? Eu gostaria de usá-lo nos meus projetos tbm, por isso que acho interessante criar esse arcabouço pra que outras pessoas possam contribuir também.

Bom, o que me diz? =) Abraço!

[lipe14-ops]

OPA @joepreludian !!!

Que bom que vc gostou do projeto... Muito boas as observações que foram feitas e sim... Estou aberto a melhorias fique a totalmente a vontade para fazer os PRs.

Obrigado :)

[joepreludian]

Que coisa boa, amigo. Claro. Irei criar PRs então para cada um dos pontos. Obrigado.

On Sat, Sep 24, 2022, 1:53 PM FilipeNS @.***> wrote:

OPA @joepreludian https://github.com/joepreludian !!!

Que bom que vc gostou do projeto... Muito boa as observações que foram feitas e sim... Estou aberto a melhorias fique a totalmente a vontade para fazer os PRs.

Obrigado :)

— Reply to this email directly, view it on GitHub https://github.com/lipe14-ops/brasilapy/issues/1#issuecomment-1257014832, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAURDNZ3CIC53HASA5LJ573V74W2NANCNFSM6AAAAAAQUVI2RY . You are receiving this because you were mentioned.Message ID: @.***>

[lipe14-ops]

Desde de já muito obrigado pela contribuição!!!