Code review comment for lp:~dooferlad/linaro-license-protection/add_do_if_old

Revision history for this message
Deepti B. Kalakeri (deeptik) wrote :

This looks good to me.
I had one suggestion, why don't we provide the option to specify the time trigger_age_string in days as well. It would be little inconvenient to convert the days into hrs if we intend to find information which are slightly very old.

otherwise +1

Thanks!!!
Deepti.

« Back to merge proposal