Skip to content
This repository has been archived by the owner on Nov 21, 2020. It is now read-only.

Generate documentation with Sphinx>=3.2 #110

Merged
merged 1 commit into from Aug 10, 2020
Merged

Generate documentation with Sphinx>=3.2 #110

merged 1 commit into from Aug 10, 2020

Conversation

McSinyx
Copy link
Owner

@McSinyx McSinyx commented Jul 23, 2020

Resolve GH-109 when Sphinx 3.2 and palace 0.2.1 are released

@McSinyx
Copy link
Owner Author

McSinyx commented Aug 8, 2020

Sphinx 3.2 seems to have some bug related to Cython:

$ make html
Running Sphinx v3.2.0
loading pickled environment... failed
failed: No such config value: c_allow_pre_v3
building [mo]: targets for 0 po files that are out of date
building [html]: targets for 17 source files that are out of date
updating environment: [new config] 17 added, 0 changed, 0 removed
reading sources... [ 35%] reference/buffer                                                                                                                                      
Extension error:
Handler <function _process_docstring at 0x7f83cc861550> for event 'autodoc-process-docstring' threw an exception (exception: module, class, method, function, traceback, frame, or code object was expected, got cython_function_or_method)
make: *** [Makefile:20: html] Error 2

Edit: I filed sphinx-doc/sphinx#8074 for this.

Edit: The bug is now fixed!

@McSinyx McSinyx marked this pull request as ready for review August 9, 2020 07:47
@McSinyx McSinyx requested a review from Huy-Ngo August 9, 2020 07:48
Copy link
Collaborator

@Huy-Ngo Huy-Ngo left a comment

Choose a reason for hiding this comment

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

LGTM, I don't see any problems here.

@McSinyx McSinyx merged commit 3339433 into gh-pages Aug 10, 2020
@McSinyx McSinyx deleted the sphinx-3.2 branch August 10, 2020 03:48
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants