Hello Nicolas,

>     (arguments
>      (list
>       #:tests? #f ; requires an attached terminal
>       ))
>     (native-inputs
>      (list emacs-ert-runner))
>
> "#:tests? #f" with a reason brings some valuable information. I simply suggest
> to keep that line.

If the default for #:tests? ever flips to #t, an explicit #:tests? #f
if overlooked, would prevent tests from running, even though everyone
might assume they are.

Until such a flip occurs, this snippet simply adds to the list of
native inputs without serving any real purpose, leading to unnecessary
use of electricity and computational resources. If the original author
intended to leave a note for future readers, a comment would be a more
suitable approach.

I'm sending you *two* patches: the first removes this redundancy, and
the second updates the emacs-popwin package.

Cheers,
Bost