openerp-brazil-team team mailing list archive

["Leonardo Santagada @ Proge.com.br" <santagada@xxxxxxxxxxxx>]

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.

+1

>
> 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.

Eu posso criar um documento se alguém quiser, mas basicamente é:

http://www.python.org/dev/peps/pep-0008/
http://google-styleguide.googlecode.com/svn/trunk/pyguide.html

e algumas ferramentas automaticas para ajudar nisso são:

http://pypi.python.org/pypi/pep8
http://pypi.python.org/pypi/pyflakes
http://pypi.python.org/pypi/pylint
http://pypi.python.org/pypi/PyChecker

> 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.

Eu concordo com a parte de passos pequenos, mas um documento para cada
refectoring não é necessário. Apenas criem um branch, façam commits
bem descritivos e depois façam um pull request.

Falando nisso eu acho o launchpad péssimo para desenvolvimento, um dia
eu vou comentar sobre migrar para o github ou bitbucket, tudo isso
seria muito mais fácil lá. essa parte de review de merge request é
fraca e fez o trabalho de revisar muito dificil, não posso comentar
direto nas linhas, não vejo o código enquanto escrevo... é bem ruim.

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

-- 
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.