Sometimes unused class or function manages to slip into code base. Static code checkers like ruff, flake8 does not have rules for detecting such globally unused code.

I tried using vulture, but it has too many false positives to have it as part of CI/CD pipeline.

I have tried to implement my own, more reliable check for global deadcode detection.

Please let me know what you think about it.

you are viewing a single comment's thread
view the rest of the comments

[–] jnovinger@programming.dev 3 points 1 year ago (5 children)

This is great! I really appreciate that it returns a sensible exit code to the shell so that it can potentially fail a build. I also like the ability to maintain a list of excluded names in pyrpoject.toml so that you don't always have to sort through false positives.

Would love to see some more output options, like maybe --quiet that simply passes/fails and returns the exit code with no output or --count that either just displays a count or includes a count in the regular output. I guess I could always pipe the output to wc -l instead of having --count.

I'll see if I can't get this integrated in our CI system at work this next week.

[–] niekas@programming.dev 2 points 1 year ago (1 children)

Thank you for your feedback! Its exactly what I was looking for 👍

I have already added --count, --quiet options and regexp pattern support for --ignore-names option. Let me know if anything else pops into your mind 😃

[–] jnovinger@programming.dev 2 points 1 year ago

Wow, nice. Thank you!

load more comments (3 replies)