[MERGE] Split up PushResult

Martin Pool mbp at canonical.com
Fri Mar 6 03:32:09 GMT 2009


Martin Pool has voted comment.
Status is now: Semi-approved
Comment:
I wonder if it would be easier for hooks and others using the result if 
it was a subclass rather than containing a branchpushresult?  Or would 
that be conceptually wrong.

In passing the way you handle report here makes me think it would be 
nice for some commands to do their io by asking the UI to display the 
result.  The result can then have a 'report' or a 'as plain text' 
method, and the text ui could call that, but other UIs could do 
something else.

For details, see: 
http://bundlebuggy.aaronbentley.com/project/bzr/request/%3C1236270730.1559.10.camel%40charis.vernstok.nl%3E
Project: Bazaar



More information about the bazaar mailing list