Reduce minimum number of pull request approvals to 1 reviewer

Issue #420 resolved
Robert Leach created an issue

USE CASE: WHAT DO YOU WANT TO DO?

Resolve pull requests in a timely manner

STEPS TO REPRODUCE AN ISSUE (OR TRIGGER A NEW FEATURE)

  1. Submit a pull request
  2. Get an approval response from the required number of reviewers according to the SCMP

CURRENT BEHAVIOR

The SCMP currently requires a minimum of 2 responses (not including the author) to a pull request to approve a merge. Yet of the 3 coders able to approve a pull request, only 2 are active and knowledgeable about the code. Pull requests hence tend to linger awhile.

EXPECTED BEHAVIOR

Since there are only 2 active coders in the project and 1 must be an author, having a requirement of 1 approval seems more efficient.

DEVELOPERS ONLY SECTION

SUGGESTED CHANGE (Pseudocode optional)

Change the SCMP to require only 1 approval response.

FILES AFFECTED (where the changes will be implemented) - developers only

SCMP

LEVEL OF EFFORT - developers only

trivial

COMMENTS

Comments (3)

  1. Log in to comment