From d78d9832edfaa1371bba2cecf07a11b46d2c6ceb Mon Sep 17 00:00:00 2001 From: Travis Brown Date: Wed, 11 Mar 2020 07:01:41 -0500 Subject: [PATCH] Add note about non-master review policy --- PROCESS.md | 9 +++++++++ 1 file changed, 9 insertions(+) diff --git a/PROCESS.md b/PROCESS.md index 400accea5c..51aa465284 100644 --- a/PROCESS.md +++ b/PROCESS.md @@ -18,6 +18,15 @@ When fixing typos, improving documentation or minor build fix only one sign-off is required (although for major edits waiting for two may be preferable). +In some cases pull requests that don't target the master branch may be +merged with a single sign-off from a maintainer (as long as they +aren't also the author of the pull request). For example, backporting +changes from master to the `scala_2.11` branch generally shouldn't +require multiple reviews. If either the pull request author or a Cats +maintainer thinks that a specific change should receive multiple +approvals even though it doesn't target the master branch, their +request should be respected. + For serious emergencies or work on the build which can't easily be reviewed or tested, pushing directly to master may be OK (but is definitely not encouraged). In these cases it's best to comment in