checklist for reviewers

Open vanessayuenn opened this issue 8 years ago • 1 comments

Currently, as a reviewer approving apps submission, I check the PR against the requirements listed in our contribution guide. The guide is super comprehensive, but a lot of the rules listed there are already automatically checked by tests as part of CI.

It will be very helpful to have a checklist of items that need human eyes & judgment. Better yet, have a bot that comments on each add app PR with said checklist so reviewers can check them off as they go!

Jan 10 '18 13:01 vanessayuenn

Better yet, have a bot

Yes! I outlined a few bot ideas in #262 and #263

Jan 10 '18 19:01 zeke