Hello Nicholas, I’ve made the necessary edits, the edited patch is attached below: Nicolas Goaziou via Guix-patches via writes: > Hello, > > Divya Ranjan via Guix-patches via writes: > >> * gnu/packages/emacs-xyz.scm (emacs-binder): New variable. > > Thank you. Some comments follow. > >> +(define-public emacs-binder >> + (package >> + (name "emacs-binder") >> + (version "20241023.1154") > > Version should be "0.5.0", as set in "binder.el". > >> + (source >> + (origin >> + (method url-fetch) >> + (uri (string-append "https://melpa.org/packages/binder-" version ".tar")) > > Guix does not use MELPA as upstream because tarballs are not reliable. > Please use Codeberg repository instead. > >> + (sha256 >> + (base32 "1nvsmmg9wwan5y38dpcpffzxxvivi4708ri64f03gwcq66g1k49m")))) >> + (build-system emacs-build-system) >> + (home-page "https://codeberg.org/divyaranjan/binder") >> + (synopsis >> + "Binder is a global minor mode to facilitate working on multiple files") >> + (description "Primarily, Binder provides a global minor mode >> binder-mode. > > "binder-mode" -> "Binder mode". > >> +This allows working with files in the current >> binder-project-directory. > > "binder-project-directory" -> "@code{binder-project-directory}". > >> +Data concerning these files is saved in a .binder.el file in the >> project directory.") > > I don’t think the last sentence is useful in the description as it > merely describes the internals of the package. I suggest to remove it. > If you absolutely want to keep it, I suggest to apply proper markup: > ".binder.el" -> "@file{.binder.el}".