RastreioBot icon indicating copy to clipboard operation
RastreioBot copied to clipboard

Melhorar a modularizacao do código

Open Bigua opened this issue 6 years ago • 3 comments
trafficstars

Diminuir o tamanho dos métodos separando-os por funcionalidade única, criando assim métodos especialistas e reutilizáveis

Bigua avatar Sep 16 '19 17:09 Bigua

Essa tarefa e muito ampla e corre o risco do PR demorar muito para ser aprovado, por afetar o projeto todo. Seria interessante abrir issues para cada modulo.

Eu ja fiz um refactoring do status.py usando IntEnum. Se abrirem essa issue eu abro o PR. :-)

maurobaraldi avatar Sep 25 '19 20:09 maurobaraldi

@maurobaraldi Espera outubro pra ganhar a camiseta e manda bala! E fique a vontade pra criar a issue ou só mandar o PR Toda contribuição é super bem vinda

GabrielRF avatar Sep 26 '19 14:09 GabrielRF

@maurobaraldi conconrdo que não tem como fazer um PR só refatorando o projeto inteiro, mas guiar por módulo não parece ser uma boa. Eu consigo ver alguns casos em que novos módulos precisam ser criados, outros excluidos ou até mergeados com outros módulos.

Essa issue que o @Bigua criou pode ser usada como um ponto em comum pra essa discussão. Daqui podem surgir issues mais específicas (oq já até aconteceu: #38) :)

rougeth avatar Sep 26 '19 15:09 rougeth