← Back to team overview

openerp-community-reviewer team mailing list archive

Re: [Merge] lp:~therp-nl/ocb-server/7.0_lp1260743_print_empty_lines into lp:ocb-server

 

Review: Needs Fixing

Thanks! This actually improves on https://code.launchpad.net/~olivier-laurent/openobject-server/6.1-bug1157048/+merge/154031, which either only works in other cases or has only worked before.

I'm hesitant about the changes in whitespace. I know that the OpenERP core dev team does not like unnecessary changes, so to increase the chances of this change being merged in upstream, I would like to ask you to remove them.

-- 
https://code.launchpad.net/~therp-nl/ocb-server/7.0_lp1260743_print_empty_lines/+merge/199108
Your team OpenERP Community Backports Team is subscribed to branch lp:ocb-server.


Follow ups

References