8000 Namespace packages v2 by zeehio · Pull Request #74 · sphinx-extensions2/sphinx-autodoc2 · GitHub
[go: up one dir, main page]
Skip to content

Namespace packages v2 #74

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
Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Namespace packages v2 #74

wants to merge 3 commits into from

Conversation

zeehio
Copy link
@zeehio zeehio commented Sep 28, 2024

I found a couple of issues with PR #69, so I decided to rebase on the latest master and fix both issues.

I would love to see this merged. Is there anything else I can do to help?

Thanks

davidselassie and others added 3 commits September 22, 2024 08:52
@davidselassie @zeehio
Support documenting namespace modules
5d9e382
@zeehio
Fix TypeError when docutils can't provide a line
564b716
@zeehio
Give yield_module what it expects
a33a594 
Otherwise I get

  File "<frozen genericpath>", line 152, in _check_arg_types
TypeError: join() argument must be str, bytes, or os.PathLike object, not 'tuple'

When I rerun sphinx-build
@rafalkrupinski
Copy link

Any chance to have this merged?

@zeehio
Copy link
Author
zeehio commented Apr 27, 2025

Hi,

I stoped using this package because I felt it lacked maintenance. I understand we all have too many things to work on and not enough time.

In case anyone is interested in pursuing the review and merge of this PR, feel free to continue without me.

Best!

@rafalkrupinski
Copy link

Yeah, last commit a year ago was some formatting and comments, and before that was late 2023.

Any viable alternatives?

@rafalkrupinski
Copy link

Just wanted to report for posterity that this patch doesn't handle namespace packages coming from multiple source directories. Otherwise works great.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@zeehio @rafalkrupinski @davidselassie
0