Re: [PATCH] Add GenomicFeatures.

[Ricardo Wurmus]

Ludovic Courtès <address@hidden> writes:

> 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.

I’ll try to configure git-send-email soon.

>> +    (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!

Pushed with the above changes and after removing the “r-repository”
property (where applicable).

Thanks!

~~ Ricardo