guix-patches
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[bug#54201] [PATCH] Add wego


From: Ludovic Courtès
Subject: [bug#54201] [PATCH] Add wego
Date: Fri, 18 Mar 2022 14:59:00 +0100
User-agent: Gnus/5.13 (Gnus v5.13) Emacs/27.2 (gnu/linux)

Hi,

Nicolas Goaziou <mail@nicolasgoaziou.fr> skribis:

>>From 84b5ea4501014800ec43b6335af6b554716f8ce0 Mon Sep 17 00:00:00 2001
> Message-Id: 
> <84b5ea4501014800ec43b6335af6b554716f8ce0.1646094084.git.mail@nicolasgoaziou.fr>
> In-Reply-To: 
> <b8ff68982388247600aa8f4771e6ccd5794acf0e.1646094084.git.mail@nicolasgoaziou.fr>
> References: 
> <b8ff68982388247600aa8f4771e6ccd5794acf0e.1646094084.git.mail@nicolasgoaziou.fr>
> From: Nicolas Goaziou <mail@nicolasgoaziou.fr>
> Date: Tue, 1 Mar 2022 01:14:04 +0100
> Subject: [[PATCH] 2/2] gnu: Add wego.
>
> * gnu/packages/weather.scm: New file.
> * gnu/local.mk: Register new file.

[...]

> +++ b/gnu/packages/weather.scm
> @@ -0,0 +1,34 @@
> +(define-module (gnu packages weather)

Please add a license header.


[...]

> +    (home-page "https://github.com/schachmat/wego";)
> +    (synopsis "Weather client for the terminal")
> +    (description "@strong{wego} is a weather client for the terminal.")

Could you expound a little bit, mentioning key features, etc.?

You can prolly remove @strong.

> +    (license isc)))

Perhaps #:prefix license: ?

Otherwise LGTM, thanks!

Ludo’.





reply via email to

[Prev in Thread] Current Thread [Next in Thread]