Oliver M. Crook

Hi @jeffreypullin Thanks for having a look - that makes sense to me and thanks for investigating. I'm happy to mark for acceptance. Let me know when submit the new...

Hi @veitveit. Thanks for the submission, could you remove the errors, before I start reviewing the package? best wishes, Olly

If you want push to Bioconductor again with an update, you need to update the package version in the description file. This will trigger a new build.

Thanks @almeidasilvaf, this was extremely pleasurable to review. The code was nicely written and the vignette was clear. There are plenty of tests and examples. - [ ] The only...

Thanks Fabricio, I'm happy to accept the package after a tiny fix. Could you update the `NEWS.md` file so it matches the Desciption file?

@bphipson thanks for the submission, I'll get around to having a look at the package shortly. I can take upto 10 days to get back to you. If you could...

Apologies for they delay if reviewing the package. I'll finished with my review this week.

Hi @bphipson This looks pretty good and the vignette was very clear. I have some suggestions for changes to the package. Description - [ ] Could you make the package...

Happy to collab on that section

Thanks will try and fix this oddly I have two issues 1) I cant reproduce the ERROR locally 2) I'm getting access denied fetching from the bioconductor git repository