-
Notifications
You must be signed in to change notification settings - Fork 128
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
dead code reported by vulture #17
Comments
Hey RJ that's a really nice project you got over there! But actually here's an issue, Stephanie calls all of their methods dynamically by reading from modules.json file! even class is instantiated dynamically, and the names differ from CamelCase to snake_case on converting from modules.json to dynamically instantiate and use the method which is triggered by certain keywords! You can learn more about it here! https://slapbot.github.io/documentation/resources/algorithm/ But still kudos to your project and hopefully it'll be amazing! But maybe Stephanie is not the ideal choice to test it! I will try to contribute to your project as well! Cheers! |
o O A great deal of work, I will be having a look!
You can try to extend this functionality to read the dynamically called function through some mapping but this would be hard and error prone I guess, but you are welcome to have a look and work on it anyways! 😄
Thank You Credits: @jendrikseipp |
Thanks, Rahul for testing vulture on Stephanie, but I agree with @SlapBot in that Stephanie's dynamic nature makes it a difficult testbed for vulture. I think it's better to test vulture on "less dynamic" projects. |
We used vulture (https://github.com/jendrikseipp/vulture) to search
for unused code in your project. You can find the report below. It
would be great if you could give us feedback about which items are
actually used or unused. This would allow us to improve vulture and
ideally it also helps you to remove obsolete code or even find typos
and bugs.
Command:
Raw Result:
There might be false positives, which can be prevented by adding them to a
whitelist file. You may find more info here
Regards,
vulture team
The text was updated successfully, but these errors were encountered: