Code review comment for lp:~brian-murray/arsenal/date_last_updated

Revision history for this message
Brad Figg (brad-figg) wrote :

On 11/29/2011 06:49 PM, Bryce Harrington wrote:
> Review: Approve
>
> I'm not super familiar with this script yet, but conceptually the new options sound sensible. +1 from me.

I'm not sure how you bzr folks normally deal with these
requests and reviews. I've emailed Brian separately with
some questions/suggestions. Overall, this is a good fix.
I'd almost prefer it to be the default rather than a
command line option but it's good either way.

Brad
--
Brad Figg <email address hidden> http://www.canonical.com

« Back to merge proposal