Skip to content

Commit f4beb49

Browse files
committed
test: Remove python dead code linter
Primarily I'd like to remove this because it is very imprecise, due to Python's dynamic nature, giving it a large list of false positives that need to be listed as exceptions. See for example #16906. It's also a frequent source of complaints. I'm doubtful of the usefulness of checking for dead code in a linter in the first place. Having some dead code in the test framework for a while is not a disaster.
1 parent 36604b4 commit f4beb49

File tree

3 files changed

+0
-70
lines changed

3 files changed

+0
-70
lines changed

ci/lint/04_install.sh

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,6 @@ export LC_ALL=C
88

99
travis_retry pip3 install codespell==1.15.0
1010
travis_retry pip3 install flake8==3.7.8
11-
travis_retry pip3 install vulture==1.0
1211

1312
SHELLCHECK_VERSION=v0.6.0
1413
curl -s "https://storage.googleapis.com/shellcheck/shellcheck-${SHELLCHECK_VERSION}.linux.x86_64.tar.xz" | tar --xz -xf - --directory /tmp/

test/lint/lint-python-dead-code-whitelist

Lines changed: 0 additions & 46 deletions
This file was deleted.

test/lint/lint-python-dead-code.sh

Lines changed: 0 additions & 23 deletions
This file was deleted.

0 commit comments

Comments
 (0)