[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