-
-
Notifications
You must be signed in to change notification settings - Fork 104
For Reviewers
Karthik Ram edited this page Apr 23, 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 in a timely manner. Comments are welcome in the same issue as a package onboarding request. If your review comes as a series of issues, please submit those directly to the authors' repository and make a note in the review issue.