Skip to content

Omit non-dir items when constructing a NamespaceReader#315

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.

Sign up for GitHub

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

Jump to bottom
Merged
jaraco merged 2 commits into from
Sep 9, 2024
Merged

Omit non-dir items when constructing a NamespaceReader #315

jaraco merged 2 commits into from
Sep 9, 2024
+29 −4

Conversation

@jaraco
Copy link
Member

@jaracojaraco commented Aug 20, 2024•
edited
Loading

Closes#311

  • Add test capturing failure when resolving the MultiplexedPath for a namespace package with non-path elements in the path.
  • Omit sentinel values from a namespace path.

@jaracojaracoforce-pushed the bugfix/311-non-path-namespace-paths branch from ae4b882 to c9a1aaeCompareAugust 20, 2024 16:14
jaraco
jaraco commented Aug 20, 2024
View reviewed changes
@jaracojaraco requested a review from FFY00August 20, 2024 16:26
@jaraco
Add test capturing failure when resolving the MultiplexedPath for a n…
47d73b1
…amespace package with non-path elements in the path. Ref #311
@jaraco
Omit sentinel values from a namespace path.
2c145c5
When editable installs create sentinels, as they are not a valid directory, they're unsuitable for constructing a `MultiplexedPath`. Filter them out. Fixes#311
@jaracojaracoforce-pushed the bugfix/311-non-path-namespace-paths branch from c9a1aae to 2c145c5CompareSeptember 9, 2024 16:56
@jaracojaraco merged commit 63a7bcb into mainSep 9, 2024
@jaracojaraco deleted the bugfix/311-non-path-namespace-paths branch September 9, 2024 17:00
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@FFY00FFY00Awaiting requested review from FFY00

1 more reviewer

@pablogsalpablogsalpablogsal left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Calling importlib_resources.files with an editable install fails

3 participants

@jaraco@pablogsal