← Back to team overview

openerp-community-reviewer team mailing list archive

Re: [Merge] lp:~camptocamp/project-reporting/7.0-fix-dissociate-lep into lp:project-reporting

 

Thanks for your review Alexandre.

1. We changed the key passed in "vals", but the key in the context is still the same. So that LGTM.

2. I followed http://www.python.org/dev/peps/pep-0257/#multi-line-docstrings "Insert a blank line before and after all docstrings (one-line or multi-line) that document a class"
-- 
https://code.launchpad.net/~camptocamp/project-reporting/7.0-fix-dissociate-lep/+merge/200695
Your team Project Core Editors is subscribed to branch lp:project-reporting.


References