The problem is a missing docstring in __init__.py

Author: jettero

.pipeline

@@ -51,8 +51,8 @@ pipeline {
                     else find hubblestack -name "*.py" -print0 | xargs -r0 git diff --name-only "$LHS" "$RHS"
                     fi > relevant-files.txt
                     '''
-                sh '''mkdir -vp tests/unittests/output
-                      cp relevant-files.txt tests/unittests/output
+                sh ''' mkdir -vp tests/unittests/output
+                       cp relevant-files.txt tests/unittests/output
                    '''
             }
         }
@@ -73,14 +73,9 @@ pipeline {
                     steps {
                         sh '''#!/bin/bash
                             source ./venv/bin/activate
-                            if grep -q . relevant-files.txt; then
-                                < relevant-files.txt  xargs -r pylint --output-format=json \
-                                    > tests/unittests/output/pylint.json
-                                x=$?
-                            else
-                                echo "[]" > tests/unittests/output/pylint.json
-                                x=0
-                            fi
+                            < relevant-files.txt  xargs -r pylint --output-format=json \
+                                > tests/unittests/output/pylint.json
+                            x=$?
                             python ./tests/automation/pylint-json-to-html tests/unittests/output/pylint.json
                             exit $x
                             '''
@@ -91,17 +86,12 @@ pipeline {
                         catchError(buildResult: 'SUCCESS', stageResult: 'FAILURE') {
                             sh '''#!/bin/bash
                                 source ./venv/bin/activate
-                                if grep -q . relevant-files.txt; then
-                                    < relevant-files.txt  xargs -r bandit -lif html \
-                                      > tests/unittests/output/bandit.html
-                                    x=$?
-                                    cp tests/static/bandit.css tests/unittests/output/bandit.css
-                                    sed -i -e '/<style/,/<.style/c<link rel="stylesheet" href="bandit.css">' \
-                                      tests/unittests/output/bandit.html
-                                else
-                                    echo "<p>did not run</p>" > tests/unittests/output/bandit.html
-                                    x=0
-                                fi
+                                < relevant-files.txt  xargs -r bandit -lif html \
+                                  > tests/unittests/output/bandit.html
+                                x=$?
+                                cp tests/static/bandit.css tests/unittests/output/bandit.css
+                                sed -i -e '/<style/,/<.style/c<link rel="stylesheet" href="bandit.css">' \
+                                  tests/unittests/output/bandit.html
                                 exit $x
                                 '''
                         }

pylintrc

@@ -373,6 +373,13 @@ enable=blacklisted-name,
        bad-thread-instantiation,
        shallow-copy-environ
 
+# This wants to require a docstring at the top of every file.  We don't always
+# want that; particularly for __init__.py files.  If there was a way to disable
+# this for only __init__.py files, that would be better as some of ours are
+# auto-generated.  Until we find a way to to that without a comment in the file
+# (which is roughly the same as a docstring), we'll do this:
+disable=missing-module-docstring
+
 
 [REPORTS]