diff options
| author | Stephen Finucane <sfinucan@redhat.com> | 2023-05-08 11:12:16 +0100 |
|---|---|---|
| committer | Stephen Finucane <sfinucan@redhat.com> | 2023-05-10 10:51:00 +0100 |
| commit | 5cb16aebcc53033ff1fd785b957eef0b7c9773e2 (patch) | |
| tree | 0b54c7eafc0a792e4da63b2ac94c4b6fb72f67b2 /tox.ini | |
| parent | 379db31ea287d38a1877711548aea9b39b6573d8 (diff) | |
| download | python-openstackclient-5cb16aebcc53033ff1fd785b957eef0b7c9773e2.tar.gz | |
Use pre-commit for 'pep8' tox target, bump versions
We want to bump the versions of hacking, but doing so requires changes
in two places: '.pre-commit-config.yaml' and 'tox.ini'. This is silly:
we can simply use tox to handle pre-commit and leave all other
dependencies to pre-commit. Do this, migrating bandit to pre-commit and
bumping the other dependencies in the process.
We remove the 'fast8' target and tool, since pre-commit is plenty fast
here.
Change-Id: Ifdc582ecc3b4479ea9ed9abcb16577c6e8b2f5c1
Signed-off-by: Stephen Finucane <sfinucan@redhat.com>
Diffstat (limited to 'tox.ini')
| -rw-r--r-- | tox.ini | 19 |
1 files changed, 4 insertions, 15 deletions
@@ -21,22 +21,11 @@ deps = commands = stestr run {posargs} allowlist_externals = stestr -[testenv:fast8] -# Use same environment directory as pep8 env to save space and install time -setenv = - VIRTUAL_ENV={envdir} -envdir = {toxworkdir}/pep8 -commands = - {toxinidir}/tools/fast8.sh - [testenv:pep8] deps = - hacking>=2.0.0 - bandit!=1.6.0,>=1.1.0 - flake8-import-order>=0.13 # LGPLv3 + pre-commit commands = - flake8 - bandit -r openstackclient -x tests -s B105,B106,B107,B401,B404,B603,B606,B607,B110,B605,B101 + pre-commit run --all-files --show-diff-on-failure [testenv:bandit] # This command runs the bandit security linter against the openstackclient @@ -76,9 +65,9 @@ commands = allowlist_externals = stestr [testenv:functional] -setenv = +setenv = OS_TEST_PATH=./openstackclient/tests/functional -passenv = +passenv = OS_* commands = stestr run {posargs} |