openerp-brazil-team team mailing list archive

[Fabio Negrini <fnegrini@xxxxxxxxx>]

Senhores,

Concordo com tudo o que foi dito até agora, porém penso que temos que andar a pequenos passos pois a localização já é código produtivo em diversas empresas e não podemos correr o risco de disponibilizar algo com erros pelo anseio de querer mudar tudo. O código que alterei foi bem localizado e pretende ajustar algo que até então não era utilizado. Como iniciei o desenvolvimento do emissor NFe, estas correções foram uma consequência disto.

Com relação às críticas sobre a forma: Poderíamos criar um documento que padronizasse os desenvolvimentos e todos os desenvolvedores tivessem que seguir. O pessoal da Proge poderia disponibilizar algo do tipo, pois pelo conteúdo da revisão que fizeram, suponho que estejam bem avançados neste assunto. Enquanto isso, proponho que validem somente a funcionalidade e não a forma do meu código neste commit para não burocratizar demais o processo.

Com relação às propostas de "refactoring": Acho todas bastante válidas, pois também sou adepto de um código cada vez mais limpo e de fácil entendimento. Entretanto, como disse no primeiro parágrafo, devemos fazer isso a pequenos passos e com mais controle. Como sugestão, um blueprint para cada refactoring seria uma forma de termos maior controle.

Por enquanto é isso, aguardo validação e aprovação caso não haja mais nenhuma crítica.

Att

Fabio Negrini

-- 
https://code.launchpad.net/~fnegrini/openerp.pt-br-localiz/openerp.pt-br-localiz/+merge/80941
Your team OpenERP Brazil Team is subscribed to branch lp:openerp.pt-br-localiz/6.0.