Re: [PATCH] piem-inboxes: Support mapping inbox to multiple coderepos

[Ihor Radchenko <yantar92@gmail.com>]

Kyle Meyer <kyle@kyleam.com> writes:

> An inbox may regularly receive patches for multiple repos.  Allow an
> inbox's :coderepo to be a list, and teach piem-inbox-coderepo to
> select a target repo by prompting the caller with the configured list.

Thanks!

> @@ -100,8 +102,7 @@ (defcustom piem-inboxes nil
>    ;; `piem--merge-config-inboxes' so that the value can also be set in
>    ;; ~/.public-inbox/config.
>    :type '(alist :key-type string
> -                :value-type
> -                (plist :value-type string))
> +                :value-type plist)

:value-type plist seems wrong, unless I miss something. Even a list of
strings will not necessarily match this value type when there are e.g. 3
repos associated with an inbox.

> -               (when-let ((coderepo
> +               (when-let ((coderepos
>                             (and (eq prop-name :coderepo)
> -                                (car (gethash
> -                                      (format "coderepo.%s.dir" (car val))
> -                                      pi-cfg)))))
> +                                (mapcar
> +                                 (lambda (v)
> +                                   (car (gethash (format "coderepo.%s.dir" v)
> +                                                 pi-cfg)))
> +                                 val))))

Does it mean that :coderepo "a single string" format is no longer supported?

Best,
Ihor