Skip to content

Commit e06d4a4

Browse files
authored
use vulture hook (#45173)
1 parent 5cca83f commit e06d4a4

File tree

3 files changed

+29
-2
lines changed

3 files changed

+29
-2
lines changed

.pre-commit-config.yaml

+7
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,13 @@ repos:
88
hooks:
99
- id: absolufy-imports
1010
files: ^pandas/
11+
- repo: https://github.com/jendrikseipp/vulture
12+
rev: 'v2.3'
13+
hooks:
14+
- id: vulture
15+
entry: python scripts/run_vulture.py
16+
pass_filenames: true
17+
require_serial: false
1118
- repo: https://github.com/python/black
1219
rev: 21.12b0
1320
hooks:

pandas/tests/frame/test_constructors.py

+1-2
Original file line numberDiff line numberDiff line change
@@ -1285,8 +1285,7 @@ def test_constructor_empty_list(self):
12851285

12861286
# Empty generator: list(empty_gen()) == []
12871287
def empty_gen():
1288-
return
1289-
yield
1288+
yield from ()
12901289

12911290
df = DataFrame(empty_gen(), columns=["A", "B"])
12921291
tm.assert_frame_equal(df, expected)

scripts/run_vulture.py

+21
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,21 @@
1+
"""Look for unreachable code."""
2+
3+
import argparse
4+
import sys
5+
6+
from vulture import Vulture
7+
8+
if __name__ == "__main__":
9+
parser = argparse.ArgumentParser()
10+
parser.add_argument("files", nargs="*")
11+
args = parser.parse_args()
12+
13+
v = Vulture()
14+
v.scavenge(args.files)
15+
ret = 0
16+
for item in v.get_unused_code(min_confidence=100):
17+
if item.typ == "unreachable_code":
18+
print(item.get_report())
19+
ret = 1
20+
21+
sys.exit(ret)

0 commit comments

Comments
 (0)