search

group2py / green-earth-backend

https://green-earth.fly.dev/
0 stars 0 forks source link

Python Lint pep 8 #5

Closed heltonteixeira92 closed 1 year ago

heltonteixeira92 commented 1 year ago

Criar setup para garantir a boa qualidade de código, recomendo flake8 ou ruff mas fique a vontade para implementar o que achar melhor.

heltonteixeira92 commented 1 year ago

Meterial de apoio: https://flake8.pycqa.org/en/latest/user/configuration.html https://pep8.org/

wpgoncalves commented 1 year ago

Criar setup para garantir a boa qualidade de código, recomendo flake8 ou ruff mas fique a vontade para implementar o que achar melhor.

Ótimo, costumo utilizar bastante o autopep8, mypy e o flake8. Inclusive faço uso destes no desenvolvimento, mas não costumo incluí-los nos requisitos, deixando-o mais limpo possível para a colocação em produção! Para não instalar requisitos desnecessários quando em produção! Acham que estes requisitos devem ser incluídos em requirements?

heltonteixeira92 commented 1 year ago

@wpgoncalves nesses casos criamos um requimentes de desenvolvimento e separamos as dependencias de dev e prod ex: requirements-dev.txt vou rodar uma pipeline de CI/DC no github actions então precisamos ter essas dependencias no projeto

wpgoncalves commented 1 year ago

@wpgoncalves nesses casos criamos um requimentes de desenvolvimento e separamos as dependencias de dev e prod ex: requirements-dev.txt vou rodar uma pipeline de CI/DC no github actions então precisamos ter essas dependencias no projeto

Ok! Posso estar providenciando mais tarde, provavelmente depois da 20h30 ou 21h por motivo de trabalho!

wpgoncalves commented 1 year ago

@heltonteixeira92 boa noite! Fiz a inclusão do arquivo requirements-dev.txt com as dependências de mypy, autopep8 e flake8 para manutenção da qualidade do código desenvolvido. Acredito que é necessário o aceito da pull request para que o conteúdo passe a integrar as alterações da main.