-
-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Use set
instead of True
-only dict
#4366
base: main
Are you sure you want to change the base?
Conversation
058f83a
to
8fcc472
Compare
…m/Avasam/setuptools into Use-set-instead-of-True-only-dict
…m/Avasam/setuptools into Use-set-instead-of-True-only-dict
Thank you very much for the continuous contributions, @Avasam
Can we live without this one? Maybe if it is a breaking change we can step back? |
Since v70 just got published, I'll make a version of this PR that doesn't touch public names (let me know if any of the 4 public names listed above were not meant as public api anyway). So this can be considered for v71 instead.
I don't think so, it's just a datatype improvement. |
…t-instead-of-True-only-dict
Summary of changes
Use
set
instead ofTrue
-onlydict
Note this is likely a breaking changes for some downstream users of
pkg_resources.ResourceManager.cached_files
setuptools.package_index.PackageIndex.package_pages
's valuessetuptools.package_index.PackageIndex.scanned_urls
setuptools.package_index.PackageIndex.fetched_urls
if they access of modify its content.
Pull Request Checklist
newsfragments/
.(See documentation for details)