serenata-de-amor icon indicating copy to clipboard operation
serenata-de-amor copied to clipboard

pep8 aaaaaaall the way

Open juliatessler opened this issue 3 years ago • 3 comments

What is the purpose of this Pull Request? Fix #546

What was done to achieve this purpose? Run isort for all files (this makes files comply with PEP8)

How to test if it really works? Just run the regular unit tests of the repo and see that nothing is broken

Who can help reviewing it? Anyone who understands the repo

TODO none

P.S.: I know this is a damn long PR. Please, let me know if you'd like me to break it and how I should partition it (:

juliatessler avatar Oct 14 '21 14:10 juliatessler

_

Oi, @juliatessler ! Muito massa esse feito :)

Queria propôr a gente continuar mantendo o código formatado. O que acha de inserir uma github action pra fazer esse check? pre-commit pra formatar automaticamente também seria uma boa...

Inclusive, não precisamos seguir o PEP8 à risca e usar algo como Black.

Me diz o que você acha e qualquer coisa posso ajudar o/


English

Hi, @juliatessler ! Very nice achievement :)

I'd like to make the proposition for us to keep the code formatted. What do you think about a github action to make this check? pre-commit to format it automatically would also be nice...

Actually, we don't need to strictly follow PEP8 and can use something like Black.

Tell me what you think and I can help if needed o/

ogecece avatar Oct 27 '21 18:10 ogecece

Oi @giuliocc!

Acho super justo, mas não tenho a menor idéia de como faz isso hahaha Tentei algo usando o autoyapf, mas não tenho muitas idéias de como testar... Se puder dar uma olhada e sugestões, agradeço muito (:

juliatessler avatar Oct 27 '21 19:10 juliatessler

@juliatessler

Aeeee! Com prazer :)

Essa live do Dunossauro com o Willian Lopes cobre bem tudo que você precisaria saber sobre o assunto (formatação de código, CI, Github Actions, etc.). Se quiser exemplos de repositórios que utilizam essas ferramentas tem a toolbox e o de raspadores do Querido Diário (que usam black, flake8 e isort).

Qualquer coisa pode chamar :)


English

Weeeew! Be my guest :)

This live from Dunossauro with Willian Lopes covers pretty much everything you would need to know about this subject (code formatting, CI, Github Actions, etc.). If you also want examples of repos that use those tools there are the Querido Diário's toolbox and spiders repos (that use black, flake8 and isort).

ogecece avatar Oct 27 '21 20:10 ogecece