From d2c8e1e567db784d27da5f6f4be1bf8ef769855d Mon Sep 17 00:00:00 2001 From: Jack Harper Date: Tue, 16 Apr 2019 15:48:54 +0100 Subject: [PATCH] Update CONTRIBUTING.md --- CONTRIBUTING.md | 2 ++ 1 file changed, 2 insertions(+) diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md index adb14411b..72625d753 100644 --- a/CONTRIBUTING.md +++ b/CONTRIBUTING.md @@ -15,6 +15,8 @@ The names should start with the ticket number and contain a brief description se ## Pull Requests There is a template for pull requests. This should contain enough information for the reviewer to be able to review the code efficiently. +In the case of an in-progress pull request, don't assign any reviewers as it can clog up their github notifications and if someone starts reviewing that's outdated it can potentially cause issues. + ## Code style All code should be pep8 compliant, we have a linter built into the CI system. [Black](https://black.readthedocs.io/en/stable/the_black_code_style.html) code style should be followed where possible unless readability is compromised.