-
-
Notifications
You must be signed in to change notification settings - Fork 104
For Reviewers
Karthik Ram edited this page Apr 27, 2015
·
24 revisions
- Does the code comply with general principles in the Mozilla reviewing guide?
- Are there improvements that could be made to the code style? ROpenSci packaging guide?
- Is there a lot of code duplication in the package?
- Are there user interface improvements that could be made?
- Are there performance improvements that could be made?
- Is the documentation (installation instructions/vignettes/examples/demos) clear and sufficient?
Also ensure that
- tests pass for you locally.
- vingettes and README examples run without issues.
- and last but not least, please be respectful and kind to the authors in your reviews.
- Please strive to complete your review within 10 days of accepting a review reqyest. Comments are welcome in the same issue as a package onboarding request. If your review comes as a series of issues (perhaps connected to a milestone), please submit those directly to the authors' repository and make a note in the review issue. We hope that package reviews will work as an ongoing conversation with the authors as opposed to a single round of reviews typical of manuscripts.