Code review comment for lp:~luc-demeyer/account-financial-report/7.0-account_financial_report_webkit-fixes
Revision history for this message
| Yannick Vaucher @ Camptocamp (yvaucher-c2c) wrote | # |
© 2004
Canonical Ltd.
•
Terms of use
•
Data privacy
•
Contact Launchpad Support
•
Blog
•
Careers
•
System status
•
67d34a1
(Get the code!)
Luc,
If you don't split this merge in 3 parts it will certainly take even more time to be reviewed.
Keep in mind that big merge proposal take longer to review and are less likely to be quick checked by other people who want to take time for that but doesn't necessarily have more time than you to spend on it.
Plus, see my previous remarks, you better have to make a new merge proposal anyway, so why not take a bit more of time to make 3?
Regards,
Yannick