savoirfairelinux-openerp team mailing list archive

Thread
Date

Re: [Merge] lp:~savoirfairelinux-openerp/openerp-program/result_indicator_conflict into lp:~savoirfairelinux-openerp/openerp-program/program

To: David Cormier <david.cormier@xxxxxxxxxxxxxxxxxxxx>
From: Joao Alfredo Gama Batista <joao.gama@xxxxxxxxxxxxxxxxxxxx>
Date: Thu, 13 Mar 2014 15:34:33 -0000
In-reply-to: <20140313151321.20479.82362.launchpad@ackee.canonical.com>
Reply-to: mp+210819@xxxxxxxxxxxxxxxxxx
Sender: bounces@xxxxxxxxxxxxx

Review: Needs Fixing code review, no tests

Hi David,

There are some minor pep8 issues in your code:

l.479.491 Missing space.

l.562 I didn't found where the result_pool variable is used.

 
-- 
https://code.launchpad.net/~savoirfairelinux-openerp/openerp-program/result_indicator_conflict/+merge/210819
Your team Savoir-faire Linux' OpenERP is subscribed to branch lp:~savoirfairelinux-openerp/openerp-program/program.

References

[Merge] lp:~savoirfairelinux-openerp/openerp-program/result_indicator_conflict into lp:~savoirfairelinux-openerp/openerp-program/program
From: David Cormier, 2014-03-13