-
Notifications
You must be signed in to change notification settings - Fork 30
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
django-memoize vs. memoize #23
Comments
Agreed. I see |
I just did that and got very confused until i found this post :-) |
That means this post can now be found via search engines :D |
This means, that |
There is a pypi package called "memoize" (which exists since at least 2013), while there also is "django-memoize" (this package). However, the module in this package is called "memoize", which is conflicting with the existing pypi package.
One could accidentally do a "pip install memoize", which would result in the following error message when you run your code:
from memoize import memoize, delete_memoized, delete_memoized_verhash ImportError: cannot import name 'memoize'
Though the worst part of this is when you consider recent squatting events on pypi, where people register "fake" packages with malicious code (see https://www.pytosquatting.org/ for more details, in case you missed it).
There is no imminent threat for django-memoize, but I would like to begin a discussion about renaming the module in this package to "django_memoize", so that it matches the package name.
Any thoughts on this from others?
The text was updated successfully, but these errors were encountered: