You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
Duplicated TraversableResourcesLoader and wrap_spec #227
8207434 seems to have mistakenly added some TraversableResourcesLoader and wrap_spec implementations to the _adapters module. Objects with the same name and better implementation already exist in _compat, and it seems to me these ones are leftover from a previous code redesign. Can you confirm @jaraco?
Waiting on #221 getting resolved to submit a PR as it will conflict.
I encountered this confusion as well while getting back up-to-speed on this project.
The key difference between the implementation in _adapters and _compat is that _adapters.* is used in CPython 3.10, where the FileReader and ZipReader and NamespaceReaders exist natively for their respective loaders. _compat is used in the backport only and exists to support older Pythons where those loaders don't provide the native implementations presenting TraversableResources.
I can update the docstrings in those modules to reflect this need.
Activity
jaraco commentedon May 23, 2021
I encountered this confusion as well while getting back up-to-speed on this project.
The key difference between the implementation in
_adaptersand_compatis that_adapters.*is used in CPython 3.10, where the FileReader and ZipReader and NamespaceReaders exist natively for their respective loaders._compatis used in the backport only and exists to support older Pythons where those loaders don't provide the native implementations presentingTraversableResources.I can update the docstrings in those modules to reflect this need.
FFY00 commentedon May 23, 2021
That clarifies it, thanks!
Update docstrings to give more clarity on the intention of _compat.Tr…