Subject: [bug#73583] [PATCH] gnu: emacs-org-ref: Update to 3.1-0.732a20b.

From: Nicolas Goaziou <[email-masked]>
Date: Tue, 15 Oct 2024 08:41:37 +0200

Hello,

Cayetano Santos via Guix-patches via <[email-masked]> writes:

> * gnu/packages/emacs-xyz.scm (emacs-org-ref): Update to 3.1-0.732a20b.

Thank you. Some comments follow.

>          #:exclude #~(list
> -                     ;; github.com/jkitchin/org-ref/issues/1085
> -                     "openalex.el"

Why are you removing these lines? There is no mention about it in your
commit message.

>                       ;; author doesn't recommend using it
>                       "org-ref-pdf.el")))
>        (propagated-inputs
> -       (list emacs-avy
> -             emacs-citeproc-el
> -             emacs-dash
> -             emacs-f
> -             emacs-helm
> -             emacs-helm-bibtex
> -             emacs-htmlize
> -             emacs-hydra
> -             emacs-ivy
> -             emacs-ox-pandoc
> -             emacs-parsebib
> -             emacs-s))
> +       (list
> +        emacs-dash
> +        emacs-s
> +        emacs-f
> +        emacs-htmlize
> +        emacs-hydra
> +        emacs-avy
> +        emacs-parsebib
> +        emacs-helm-bibtex
> +        emacs-citeproc-el
> +        emacs-ox-pandoc
> +        emacs-request))

I don't understand the purpose of this change either. You're breaking
alphabetic order and use a dubious indentation.

Regards,
-- 
Nicolas Goaziou