← Back to team overview

openerp-community-reviewer team mailing list archive

Thread
Date

Re: lp:~pedro.baeza/account-financial-report/6.1-balance-reporting into lp:account-financial-report/6.1

To: Pedro Manuel Baeza <pedro.baeza@xxxxxxxxx>
From: "Maxime Chambreuil \(http://www.savoirfairelinux.com\)" <maxime.chambreuil@xxxxxxxxxxxxxxxxxxxx>
Date: Sun, 10 Nov 2013 16:27:54 -0000
In-reply-to: <20130830095943.16863.84026.launchpad@ackee.canonical.com>
Reply-to: mp+183114@xxxxxxxxxxxxxxxxxx
Sender: bounces@xxxxxxxxxxxxx

Review: Needs Fixing code review

Hello Pedro,

Please fix PEP8 issues given by flake8 and consider translating the spanish comments in the code (l257).

Otherwise LGTM.

Thanks!
-- 
https://code.launchpad.net/~pedro.baeza/account-financial-report/6.1-balance-reporting/+merge/183114
Your team Account Report Core Editors is subscribed to branch lp:account-financial-report/6.1.