vulture icon indicating copy to clipboard operation
vulture copied to clipboard

False positive example should be updated

Open yoichi opened this issue 2 years ago • 1 comments

Follow the instruction in README.md with vulture >= 2.1 results:

$ vulture dead_code.py
dead_code.py:1: unused import 'os' (90% confidence)
dead_code.py:8: unused variable 'message' (60% confidence)

It does not detect

dead_code.py:4: unused function 'greet' (60% confidence)

as expected since https://github.com/jendrikseipp/vulture/pull/219 eliminates the false positive.

I think it would be good to update the example to something that can still be reproduced.

yoichi avatar Jan 22 '22 00:01 yoichi

Thanks for the report, @yoichi! I'll get a fix up for this.

RJ722 avatar Jan 22 '22 18:01 RJ722

Fixed now. Thanks for the report!

jendrikseipp avatar Jan 08 '23 18:01 jendrikseipp