Code review comment for lp://qastaging/~indicator-applet-developers/indicator-messages/consolidate
Revision history for this message
| Sebastien Bacher (seb128) wrote | # |
© 2004
Canonical Ltd.
•
Terms of use
•
Data privacy
•
Contact Launchpad Support
•
Blog
•
Careers
•
System status
•
f361718
demo site
(Get the code!)
> Hi Sebastien, I thought I did reuse the existing branch? (lp:~indicator-applet-developers/indicator-messages/consolidate)
Sorry, I got confused by launchpad, it listed only one commit (411) on the mp with a non trivial diff, I guess it's a side effect of the reverts...
The fix in r411 seems fine to me, I'm going to build that branch/run it for a bit and comment back