Code review comment for lp://qastaging/~popey/ubuntu-terminal-app/add-control

Revision history for this message
Niklas Wenzel (nikwen) wrote :

Thank you for looking into this, Alan! :)

I haven't looked at the code myself yet, but please change the commit message to something that you would want to see in the bazaar history. "Adding commit message as required by fussy jenkins bot" probably won't tell you two years down the road what you implemented in this MP. ;)

The text you put into the description looks fine for me for example. ;)

« Back to merge proposal