Subject: Re: [bug#58660] [PATCH v2 1/3] etc: teams: Add 'show' subcommand.

From: zimoun <[email-masked]>
Date: Mon, 21 Nov 2022 17:00:52 +0100

Hi Mathieu,

On Fri, 18 Nov 2022 at 18:29, Mathieu Othacehe <[email-masked]> wrote:

> This procedure is starting to get a little complex. We could put the
> above is a sub-procedure called "print-team".

I will try to revamp for a v3.

>> +       (sort
>> +        (hash-map->list (lambda (key value) value) %teams)
>> +        (lambda (team1 team2)
>> +          (string<? (symbol->string (team-id team1))
>> +                    (symbol->string (team-id team2))))))))
>
> That could be in a variable called %sorted-teams.

This had been introduced by 18fb7c0f3078216bf4fc865d86b3be76ddc68414.
Indeed, it could be extracted; part of the revamp.

Cheers,
simon