[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] Add GenomicFeatures.
From: |
Ludovic Courtès |
Subject: |
Re: [PATCH] Add GenomicFeatures. |
Date: |
Fri, 18 Dec 2015 17:26:50 +0100 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/24.5 (gnu/linux) |
Ricardo Wurmus <address@hidden> skribis:
> PS: If reviewing these patch sets is uncomfortable in this format and
> you’d prefer separate emails or some other approach, please let me
> know. I appreciate your reviews a lot!
I have a slight preference for one message per patch, but this works for
me too.
[...]
> + (synopsis "Options management framework")
> + (description
> + "This package provides a scoped options management framework.")
Would be nice to expound a little bit (are we talking about command-line
options? what does “scoped” mean?).
> + (home-page "http://cran.r-project.org/web/packages/snow")
> + (synopsis "Support for simple parallel computing in R")
> + (description
> + "This package provides support for simple parallel computing in R.")
Likewise.
Apart from that, it looks good to me!
Ludo’.