Skip to content
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

DM-27060: pipe_base documentation fails to build due to BuildId #151

Merged
merged 1 commit into from Oct 6, 2020

Conversation

jdswinbank
Copy link
Contributor

These types cause a documentation build failure:

FileNotFoundError: [Errno 2] No such file or directory: '/j/ws/sqre/infra/documenteer/doc_template/_build/doctree/py-api/lsst.pipe.base.BuildId.doctree'

Presumably because Sphinx/autodoc doesn't understand that they are generated
by typing.NewType rather than being fully-fledged classes.

These types cause a documentation build failure:

  FileNotFoundError: [Errno 2] No such file or directory: '/j/ws/sqre/infra/documenteer/doc_template/_build/doctree/py-api/lsst.pipe.base.BuildId.doctree'

Presumably because Sphinx/autodoc doesn't understand that they are generated
by typing.NewType rather than being fully-fledged classes.
Copy link
Member

@TallJimbo TallJimbo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Telling Sphinx to ignore these seems like the best fix for now.

But if these types are causing problems because they're being used to document (e.g.) function arguments, I might ask @natelust to open a ticket to change that, too, because having docs reference a pseudo-type that can only be understood by looking at the code doesn't seem like good practice. Using NewType in type annotations is okay; using it in docstrings apparently isn't right now.

Thanks for tracking this down!

@jdswinbank jdswinbank merged commit 0d72be9 into master Oct 6, 2020
@timj timj deleted the tickets/DM-27060 branch April 13, 2022 22:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants