Code review comment for lp:~altair-ibn-la-ahad/ubuntu/precise/update-notifier/bug-887650

Revision history for this message
Andreas Altaïr Redmer (altair-ibn-la-ahad) wrote :

Hello Michael, thanks for you feedback.

Last time I relpied to Dmitry only in the BUG and not here in the MP.

I've already created this new branch

https://code.launchpad.net/~altair-ibn-la-ahad/update-notifier/bug-887650

Did I do that correctly? So I think that step is done. I have also created a MP there. Maybe we can delete this MP somehow, because it is "outdated" ?! Sorry, I am new, maybe I should have changed this MP somehow.

I have also added your new changes to that new branch, so it looks like professional code now :)

http://bazaar.launchpad.net/~altair-ibn-la-ahad/update-notifier/bug-887650/revision/783

« Back to merge proposal