From 220a3ced6ded670ff48a651d7ccb54df1a52278b Mon Sep 17 00:00:00 2001 From: Jeremy Stanley Date: Mon, 26 Sep 2022 19:58:19 +0000 Subject: [PATCH] Ignore bashate E010 The check for 'The "do" should be on same line as for' is not a good rule when the loop condition is too long to fit on a single line, so excludes more readable ways of constructing the script. Ignore it. Also, ironically, wrap some long lines in the tools/run-bashate.sh script since we're editing it anyway. Change-Id: I1f2cf44896a0137fcb954f67005268faa7291d5e --- tools/run-bashate.sh | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/tools/run-bashate.sh b/tools/run-bashate.sh index fd6328812b..e57e734c82 100755 --- a/tools/run-bashate.sh +++ b/tools/run-bashate.sh @@ -1,4 +1,6 @@ #!/bin/bash ROOT=$(readlink -fn $(dirname $0)/.. ) -find $ROOT -type f -not -wholename \*.tox/\* -and \( -name \*.sh -or -name \*rc -or -name functions\* \) -print0 | xargs -0 bashate -i E006 -v +find $ROOT -type f -not -wholename \*.tox/\* -and \( -name \*.sh \ + -or -name \*rc -or -name functions\* \) -print0 \ + | xargs -0 bashate -i E006,E010 -v