guix-devel
[Top][All Lists]
Advanced

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

Re: [PATCH 01/19] gnu: Add r-assertthat.


From: Ricardo Wurmus
Subject: Re: [PATCH 01/19] gnu: Add r-assertthat.
Date: Sun, 13 Sep 2015 22:04:47 +0200

Thank you very much for these patches!  Did you use the CRAN importer
for them?

You should add your name to the copyright header at the top of the file
(maybe as part of the first patch).

> * gnu/packages/statistics.scm (r-assertthat): New variable.

[...]

> +              (uri (string-append "mirror://cran/src/contrib/assertthat_"
> +                                  version ".tar.gz"))

I wonder if new patches should already contain the archive URI in
addition to the default URI.  It was pointed out in a previous thread
that the simple URI produced by the importer is only valid for the
latest version.

> +    (description
> +     "Assertthat is an extension to stopifnot() that makes it easy to
> +declare the pre and post conditions that you code should satisfy,

s/you code/your code/

> +while also producing friendly error messages so that your users know
> +what they've done wrong.")

~~ Ricardo




reply via email to

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