Code review comment for lp:~spiv/bzr/per-file-merge-hook-491711

Revision history for this message
Robert Collins (lifeless) wrote :

I'd suggest one of:
 return value indicates 'handled' or 'not handled'
 return value is parameter to pass to next hook or None-to-stop

and let changes that require calling fn's on Merger be done directly.

-Rob

« Back to merge proposal