Remove dead code - Githubissues

zetonteam / zeton_django

Hacktoberfest

Apache License 2.0

10 stars 2 forks source link

Remove dead code #82

Closed MartaSien closed 20 hours ago

MartaSien commented 1 month ago

There may be unused code within the repository - it should be verified and, if necessary, removed.

[x] Try out https://pypi.org/project/vulture/
[x] Try to create a Django-oriented configuration for Vulture and test it again

szmktk commented 2 weeks ago

I'll just run https://pypi.org/project/vulture/ and see what happens

EDIT: nah, it does not recognize variables & fields used by Django :face_with_diagonal_mouth:

MartaSien commented 2 weeks ago

But a good idea to use an automated solution. Perhaps there is one that works for our case. 🙂

According to this article, there is a way to configure Vulture better for Django projects, by excluding certain decorators and files.

pyexplorerhs3 commented 1 week ago

❯ vulture . | grep 100\%
tests/conftest.py:6: unused variable 'django_db_setup' (100% confidence, 1 line)
users/views.py:40: unused variable 'format' (100% confidence, 1 line)
users/views.py:180: unused variable 'format' (100% confidence, 1 line)