← Back to team overview

openerp-community-reviewer team mailing list archive

Re: [Merge] lp:~initos.com/account-analytic/7.0 into lp:account-analytic

 

Indeed, there are modules that are already hosted on community repositories that have less coding conventions than yours, but these modules were included blindly when we start the project to have a base where we can work, and for now what we are trying to do is to increase more and more the quality of new additions. In the same way, when someone propose a change on that existing modules, we ask for a refactorization of them at the same time.

I hope you understand it.

On flake8 output, you can ignore the two line of "imported but unused"

Regards.
-- 
https://code.launchpad.net/~initos.com/account-analytic/7.0/+merge/195362
Your team Account Core Editors is subscribed to branch lp:account-analytic.


References