openerp-brazil-team team mailing list archive

["Vinicius Dittgen - Proge.com.br" <vinicius@xxxxxxxxxxxx>]

Oi Fábio,

Para validar somente a funcionalidade, nós não precisaríamos olhar o
código, somente validar entradas e saídas em algum caso de teste. E desta
forma não precisaria de revisão.

Acho que precisamos começar urgentemente a escrever testes para a
localização, eu estou escrevendo alguns testes para a parte fiscal, passo o
link hoje a tarde.

Cada vez estamos criando mais códigos e para manter é melhor que tenha
revisão quanto ao formato tbm, se não cada pedaço de código ficara de uma
forma, é fundamental ao menos seguirmos a PEP8 quanto ao formato.

Apreciamos muito a tua iniciativa e queremos ajudar no que for possível
para que tu desenvolva o maior número possível de funcionalidades.

Grande Abraço


Em 8 de novembro de 2011 12:53, Fabio Negrini <fnegrini@xxxxxxxxx> escreveu:

> 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
> You are subscribed to branch lp:openerp.pt-br-localiz/6.0.
>



-- 

Att.:
-- 
Vinicius Dittgen
Proge Informática
Soluções em Gestão Empresarialhttp://www.proge.com.br
Tel.:(51)3085 0096 Cel.:(51) 9193 2020
Skype: vdittgen
Canal IRC: #openerp-br




Seja um voluntário
www.centrorepensar.com.br
Centro especializado em Deficiência Intelectual, Autismo, Síndrome de
Down e Dificuldade de Aprendizagem

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.