Merge pull request #5549 from donaldsharp/automated

doc: Update workflow to include new automated testing Requirement

diff --git a/doc/developer/workflow.rst b/doc/developer/workflow.rst
index 40378f02195c50661318a5ab8bf6ec0e84624cb2..8ce3bdeeb27e2052bd7e8833129b6c8077f8e15a 100644 (file)
--- a/doc/developer/workflow.rst
+++ b/doc/developer/workflow.rst
@@ -283,7 +283,10 @@ Pre-submission Checklist
    -  ``make test``
 
 - In the case of a major new feature or other significant change, document
-  plans for continued maintenance of the feature
+  plans for continued maintenance of the feature.  In addition it is a
+  requirement that automated testing must be written that exercises
+  the new feature within our existing CI infrastructure.  Also the 
+  addition of automated testing to cover any pull request is encouraged.
 
 .. _signing-off:
 
@@ -452,6 +455,10 @@ Guidelines for code review
   may originate with a reviewer or document agreement reached on Slack,
   the Development mailing list, or the weekly technical meeting.
 
+- Reviewers may ask for new automated testing if they feel that the
+  code change is large enough/significant enough to warrant such
+  a requirement.
+
 
 Coding Practices & Style
 ========================