We have a lots of long heredocs lines in the OCF scripts and older bashate
versions consider these as E006 violations.
From version 0.5.0, bashate doesn't check heredocs, so we specify this
version as a dependency.
In addition, this commit turns on E006 violation checking again.
Change-Id: I1ff675dd587239f0b7fd65c15b8df57a39a2c72b
Signed-off-by: Norbert Illes <norbert.e.illes@ericsson.com>
The currently available bashate releases are considering heredocs as
normal code lines, hence lines longer than 79 columns in these sections
are also considered as E006 violations. As the OCF scripts are
containing lots of heredocs, we are affected by this behaviour.
However, there is a commit in the bashate repository (649c7dc79948)
which modifies bashate to ignore long lines in heredocs.
Currently there is no bashate release which contains the above commit,
so we ignore E006 errors until a new bashate released.
Change-Id: I33a9737ce1ec7eddab0b24ddedefe5c17da03b7a
Partial-Bug: #1550203
Signed-off-by: Norbert Illes <norbert.e.illes@ericsson.com>
This commit moves the syntax-check test from a make target to tox.ini
Change-Id: Id15320c589afea2b3a4a5cff5e7fa9c5c2b9d0b8
Signed-off-by: Norbert Illes <norbert.e.illes@ericsson.com>
This commit implements a simple tox.ini configuration to run bashate
style checker against all files in the ocf directory.
Partial-Bug: #1508559
Change-Id: I34b3fc108a86d902d0d856f632b5221e14f1f118
Signed-off-by: Norbert Illes <norbert.e.illes@ericsson.com>
Norbert Illes