Re: [nongnu] elpa/nix-mode 91a317e425 2/2: Merge pull request #176 from lafrenierejm/reformatter

[Basil L. Contovounesios]

ELPA Syncer [2023-01-12 13:59 -0500] wrote:

> branch: elpa/nix-mode
> commit 91a317e425be15b03d31d404d798c21faf373113
> Merge: 54e5626829 fb6c1ca5dd
> Author: Matthew Bauer <mjbauer95@gmail.com>
> Commit: GitHub <noreply@github.com>
>
>     Merge pull request #176 from lafrenierejm/reformatter
>     
>     Replace custom `nixfmt` wrapping with `reformatter`
[...]
> diff --git a/nix-format.el b/nix-format.el
> index 75e679e46e..679ad02629 100644
> --- a/nix-format.el
> +++ b/nix-format.el
> @@ -3,54 +3,29 @@
>  ;; This file is NOT part of GNU Emacs.
>  
>  ;; Homepage: https://github.com/NixOS/nix-mode
> -;; Package-Requires: ((emacs "24.1"))
> +;; Package-Requires: ((emacs "24.1") (reformatter "0.6"))

Just curious: is it okay in this case for a NonGNU ELPA package to
depend unconditionally on a MELPA package?  Maybe reformatter can/should
also be distributed via NonGNU ELPA?

Thanks,

-- 
Basil