Subject: Re: [bug#53726] [PATCH] gnu: Add emacs-popup-kill-ring

From: Nicolas Goaziou <[email-masked]>
Date: Sun, 06 Feb 2022 22:54:57 +0100

Hello,

Jai Vetrivelan <[email-masked]> writes:

> Subject: [PATCH] gnu: Add emacs-popup-kill-ring.

Thank you.

> * gnu/packages/emacs-xyz.scm: (emacs-popup-kill-ring) new variable.

Nitpick: the colon should come after the closing bracket.

> +(define-public emacs-popup-kill-ring
> +  (let ((version "0.2.8") (revision "0")
> +        (commit "a45c9cf79c4e1207fdbdd662e161154ef224ad61"))
> +    (package
> +      (name "emacs-popup-kill-ring")
> +      (version (git-version version revision commit))

I inserted the version string above.

> +      (description "Browse your Emacs kill-ring in an autocomplete style popup
> + menu.")

I reworded the description so it would be a complete sentence.

Regards,
-- 
Nicolas Goaziou