electron / electron/apps

checklist for reviewers

Open
#461 1 comment 0 reactions 0 assignees View on GitHub
Dominant language
JavaScript
Stars
1.7k
Forks
1.5k
Avg merge
18m
Merged PRs (30d)
2

Description

Currently, as a reviewer approving apps submission, I check the PR against the requirements listed in our [contribution guide](https://github.com/electron/electron-apps/blob/master/contributing.md). The guide is super comprehensive, but a lot of the rules listed there are already automatically checked by [tests](https://github.com/electron/electron-apps/blob/master/test/human-data.js) 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!

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.