Code review comment for lp:~fernandezm/ubuntu/utopic/casper/toram-casper

Revision history for this message
Luke Yelavich (themuso) wrote :

Thanks for your work.
I am not currently in a position to test this fix to make sure it still works in all use cases, but code wise it looks ok. I would ask that you add a debian/changelog entry, and explain clearly what this change is, and why it is being made, i.e something similar to what you wrote in the merge proposal message. Your bzr log message could be used, but it is not detailed enough as to the change, and why it is being made.

Thanks again.

review: Needs Fixing

« Back to merge proposal