From f710b36cf1c0d013b116d86a31e8c993fd71c8f4 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Herv=C3=A9=20Beraud?= <hberaud@redhat.com>
Date: Thu, 7 May 2020 12:48:32 +0200
Subject: [PATCH] Ignore a struture error (E010) on bashate

It could be less annoying to ignore the E010 [1] and allow us to don't fight
against base syntaxe error with "for do" statements by avoiding
semicolon.

Also these changes add some comments about the ignored rules.

[1] https://docs.openstack.org/bashate/latest/readme.html#structure-errors

Change-Id: Ic16027bf97eb93d25fbfed21843f9df855571fa9
---
 tox.ini | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/tox.ini b/tox.ini
index 7c5c075..c6719ed 100644
--- a/tox.ini
+++ b/tox.ini
@@ -18,13 +18,15 @@ commands = flake8
 [testenv:bashate]
 deps = bashate
 whitelist_externals = bash
+# E010: do not on the same line as for
+# E006: check for lines longer than 79 columns
 commands = bash -c "find {toxinidir}          \
          -not \( -type d -name .?\* -prune \) \
          -type f                              \
          -not -name \*~                       \
          -not -name \*.md                     \
          -name \*.sh                          \
-         -print0 | xargs -0 bashate -i E006 -v"
+         -print0 | xargs -0 bashate -i E006,E010 -v"
 
 [testenv:venv]
 commands = {posargs}