]> git-server-git.apps.pok.os.sepia.ceph.com Git - ceph-build.git/commitdiff
ceph-docker-lint: exclude codes
authorSébastien Han <seb@redhat.com>
Fri, 12 May 2017 08:08:42 +0000 (10:08 +0200)
committerSébastien Han <seb@redhat.com>
Fri, 12 May 2017 08:50:46 +0000 (10:50 +0200)
Excluding the following code:

* SC1091: since our entrypoint copies files in / during this shellcheck
the files can not be found. Basically we source ./file but presently the
file is under osd_directory/file so can not be found.
* SC2015: because we use 'set -e' we don't have other choice to use '||
true' here.
* SC2009: pgrep returns processes ID where we are looking for a specific
string in the process line.
* SC2001: our sed expressions are too complex we can not substitute them
with bash ${variable//search/replace}.

Signed-off-by: Sébastien Han <seb@redhat.com>
ceph-docker-lint/build/build

index 8b3a173b46d53714899112f1d41f108b876608ad..6add50d396f2f9f196d6f70c61ef8cfd0db518ee 100755 (executable)
@@ -3,8 +3,7 @@
 set -e
 set -x
 
-#IGNORE_THESE_CODES="SC2005,SC2068,SC2086,SC2148,SC2162"
-IGNORE_THESE_CODES=""
+IGNORE_THESE_CODES="SC1091,SC2015,SC2009,SC2001"
 
 function generate_filelist(){
    if [[ "$pull_request_id" -eq "" || "${ghprbCommentBody:-}" = "jenkins lint all" ]]