← Back to team overview

openerp-community-reviewer team mailing list archive

Re: [Merge] lp:~lin-yu/purchase-wkfl/add_purchase_price_list into lp:purchase-wkfl

 

Review: Needs Fixing code review

Thanks for your contribution Yu.

l163: please use orm.Model instead of osv.osv. Update the import accordingly.
l208, 463: no need to instantiate
l417: 2 lines are enough to comply with PEP8. Run flake8 to check for PEP8 issues
-- 
https://code.launchpad.net/~lin-yu/purchase-wkfl/add_purchase_price_list/+merge/180796
Your team Purchase Core Editors is subscribed to branch lp:purchase-wkfl.