openstack/releases
Code Issues Proposed changes

Ignore line length for bashate

Browse Source 
I am not sure we really need to care about a line length that's 90
chars instead of 79. We have so many ugly things here, I am not
sure this will impact positively the readability to force line breaks.

Without this patch, all the shell scripts changes will fail with
a bashate failure, due to the fact that the
recently introduced changes in tools/make_missing_releases.sh
have lines of length above 79.

The alternative would be to fix those lines. There are only 3 of them.

Change-Id: I5fc7bde3c60c50ae78e3c7f297f715a7591e486c
This commit is contained in:
Jean-Philippe Evrard
2020-01-22 16:10:11 +01:00
parent b11035aea0
commit 96c324ba73
1 changed files with 1 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
tox.ini
View File

@@ -53,7 +53,7 @@ commands = bash -c "find {toxinidir} \
-not -name \*~ \ -not -name \*~ \
-not -name \*.md \ -not -name \*.md \
-name \*.sh \ -name \*.sh \
-print0 | xargs -0 bashate -v" -print0 | xargs -0 bashate -i E006 -v"
[testenv:aclmanager] [testenv:aclmanager]
commands = python {toxinidir}/tools/aclmanager.py {posargs} commands = python {toxinidir}/tools/aclmanager.py {posargs}