Proposal for the onboarding guide:
- Submit your code for review by us, or have someone review the code using our guide, for every 400 lines of new code in the
R/and `src/’ directories.
- After each review, consider whether the package is ready to release as-is.
The latter point is to encourage early release of well-reviewed, tested, and documented packages over waiting for feature-complete packages. Not everything is ready for release at 400 lines, but it seems like a good stage to consider it. Lines of code are of course a rough metric, but this encourages a regular check-in.
I note that rdrop2 was at ~800 lines when I reviewed it. geojsonio will be ~1500.
I added some more guidelines to the reviewers guide, too.