tags 835653 + pending thanks
Hey Nicolas,
On 28/08/16 17:31, Nicolas Braud-Santoni wrote:
> I wrote up a quick patch that takes into account the package names
> when sorting (or bug id, in the case of RFSes), then added the
> removal time (in the case of autorm) and cleaned up a bit the
> surrounding code by using `sort_by` rather than `sort { |a,b| expr(a)
> <=> expr(b) }`.
Thanks for the patches! :)
I've applied both 0001 and 0002, so #835653 should be resolved with the
upcoming upload.
However, I do agree with Paul about making the sort by removal date
feature (patch 0003) optional. I'm also wondering if sorting by removal
date as the primary sorting field, wouldn't be more useful in this case
(so that the packages that are going to be removed first are at the
top)? What do you think?
Regards,
T. Nitecki
signature.asc
Description: OpenPGP digital signature
