-
-
Notifications
You must be signed in to change notification settings - Fork 6
More backports #29
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
Comments
Thanks for this! I've only been adding things to the exclusion list when I've seen them show up as one the 360 here, and I see that the most downloaded of these ones (DateTime) is currently at 632 on Top PyPI Packages, and the next one (logging) is at 791, so at least they're unlikely to be relevant for a while. 6396758 - DateTime I'm not aiming for a definitive list of exclusions, only those that appear in the 360, so this is useful for future reference. If you want, feel free to create a PR to exclude some that are definitely backports, although if they may never show up here I wouldn't put too much effort into that. Thanks again! |
Ah, I am using the full list for my current analysis. The problem is still relevant to https://github.com/hugovk/top-pypi-packages I guess. For completeness, some other packages using stdlib names that look questionable / not backports, but need to recheck
The following have urls and no artifacts on PyPI
numbers, select and signal point to github repos which are 404 select was renamed to https://github.com/Jaymon/que time has no urls or artifacts |
I'm keeping the list at https://github.com/hugovk/top-pypi-packages vanilla with no changes at all, so people can use it as they wish. For example, maybe to find out how popular a backport is to decide whether to continue maintaining it. But thank you! |
The list of backports can be partially obtained using pypi/stdlib-list#28
Using that, I found the following are missing from the
EXCLUDED_PACKAGES
list.DateTime
so doesnt conflict withdatetime
.pprint
,resource
andast
should be excluded IMO, as I am sure their inclusion in the top packages is only due to having the same name as a stdlib package. I think they should also be delisted from PyPI and other existing stdlib names prevented from being used (or an extra permission needed to control who can use those names) to prevent malicious uploads.DateTime
would probably not be on the list if it wasnt for the stdlib name clash. Oddly, the 'used by' on https://github.com/zopefoundation/DateTime is quite high (7.2k) - I wonder if the GitHub stats are also skewed. But it could be that the GitHub stats are correct, as this is Zope, and IMO it isnt worth delisting it from this project due to the Zope aspect - perhaps intentionally move it to the end of the list since its true relevance based on download count is highly suspect, so its appropriate position in the list is not knowable.fwiw, the other stdlib names in the top list all appear to be 'safe'.
The text was updated successfully, but these errors were encountered: