openstack
/
cinder
Code Issues Proposed changes

Run pylint tox env on all files by default 

The pylint tox env and job currently only run
on the files changed in HEAD.  This is confusing,
because changes regularly merge which introduce new
pylint errors, which will not show up in a pylint
run until another patch touches that file.

Instead, just run pylint on all of Cinder to get
consistent output.

Depends-On: Icb4c0b5cbba02c42e9e2ee381dc9152f1c1de22c
Depends-On: I3e17c6a2a7f252a9d324a1fea616e22869e55702
Change-Id: I98fd7f76e5d3d096970d6c5f1652e9ee2abeebd6
This commit is contained in:
Eric Harney 2022-04-27 12:35:08 -04:00
parent b1ed75aa48
commit 91085df702
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
tox.ini
View File

@ -95,7 +95,7 @@ deps = -r{toxinidir}/requirements.txt
-r{toxinidir}/test-requirements.txt
pylint==2.11.1
commands =
{toxinidir}/tools/coding-checks.sh --pylint {posargs}
{toxinidir}/tools/coding-checks.sh --pylint {posargs:all}
[testenv:cover]
# Also do not run test_coverage_ext tests while gathering coverage as those