Subject: Re: [bug#58057] [PATCH] gnu: Add emacs-macrostep-geiser.

From: Nicolas Goaziou <[email-masked]>
Date: Sun, 25 Sep 2022 15:22:07 +0200

Hello,

Hilton Chain via Guix-patches via <[email-masked]> writes:

> * gnu/packages/emacs-xyz.scm (emacs-macrostep-geiser): New variable.

Thank you. I applied the patch with the following changes:

> +(define-public emacs-macrostep-geiser
> +  (let ((commit "f6a2d5bb96ade4f23df557649af87ebd0cc45125")
> +        (revision "1"))
> +    (package
> +      (name "emacs-macrostep-geiser")
> +      (version (git-version "0.2.0" revision commit))

I downgraded the commit used to the one matching the version bump. As
a consequence, version is exactly "0.2.0", which is always nicer when
subsequent changes do not bring much to the table.

> +      (propagated-inputs (list emacs-macrostep emacs-geiser))

I ordered inputs alphabetically.

> +      (synopsis "macrostep for geiser and cider")
> +      (description
> +       "This plug-in implements a macrostep back-end powered by geiser.")

I used capitalization for Macrostep and Geiser.

Regards,
-- 
Nicolas Goaziou