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

DOC Use sphinx < 3.0 when building docs on readthedocs #399

Merged
merged 2 commits into from Apr 21, 2020

Conversation

drewejohnson
Copy link
Collaborator

Sphinx 3.0 was recently released and between that release, the matplotlib plot_directive (also with a new release recentlyish), and the readthedocs builder, there were some warnings / errors that caused the build to fail

The fix was two fold. First, pinning our sphinx version in the docs requirements files to below 3.0 was the main fix. Then, for completeness, I fixed some minor documentation issues that caused warnings to be raised. These can become errors when building in a "strict" mode, e.g. sphinx-build -W . _build will turn warnings to errors

Related to readthedocs/readthedocs.org#6916
where some warnings in sphinx 3.0 cause the build system to warn out and fail
Anaconda prompt -> Anaconda Prompt

Use parameters block, not attributes block in UnivTuple
doc. Remove additional logic for py2 compatibility
@drewejohnson drewejohnson added this to the 0.9.3 milestone Apr 21, 2020
@drewejohnson drewejohnson added this to Review in Main Apr 21, 2020
@drewejohnson drewejohnson merged commit ffe6486 into master Apr 21, 2020
drewejohnson added a commit to drewejohnson/serpent-tools that referenced this pull request May 23, 2020
Backport of features included since 0.9.3 release

Includes the following PRs

git log --online --pretty="%s" --merges 0.9.3...

Merge pull request CORE-GATECH-GROUP#407 from drewejohnson/fix-readme
Merge pull request CORE-GATECH-GROUP#406 from drewejohnson/r0.9.3
Merge pull request CORE-GATECH-GROUP#405 from nicoloabrate/master
Merge pull request CORE-GATECH-GROUP#404 from drewejohnson/update-git-attr
Merge pull request CORE-GATECH-GROUP#402 from drewejohnson/xsplot-labels
Merge pull request CORE-GATECH-GROUP#400 from drewejohnson/lib2-notes
Merge pull request CORE-GATECH-GROUP#399 from CORE-GATECH-GROUP/docs-sphinx-2.0
Merge pull request CORE-GATECH-GROUP#397 from drewejohnson/port-list-of-arrays
Merge pull request CORE-GATECH-GROUP#398 from drewejohnson/det-supers
Merge pull request CORE-GATECH-GROUP#396 from drewejohnson/df-arg-renaming
Merge pull request CORE-GATECH-GROUP#394 from drewejohnson/0.9.3-update-docs
Merge pull request CORE-GATECH-GROUP#395 from drewejohnson/hist-branch-dictlike
Merge pull request CORE-GATECH-GROUP#393 from drewejohnson/less-metadata
Merge pull request CORE-GATECH-GROUP#392 from drewejohnson/better-access
Merge pull request CORE-GATECH-GROUP#391 from drewejohnson/dep-dataframe
Merge pull request CORE-GATECH-GROUP#390 from drewejohnson/api-dep-attributes
Merge pull request CORE-GATECH-GROUP#388 from drewejohnson/xsplot-branch
Merge pull request CORE-GATECH-GROUP#378 from drewejohnson/better-build
Merge pull request CORE-GATECH-GROUP#376 from drewejohnson/unbundle-data-files
Merge pull request CORE-GATECH-GROUP#370 from drewejohnson/results-loa
Merge pull request CORE-GATECH-GROUP#369 from drewejohnson/merge-0.9.1-develop
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Main
Review
Development

Successfully merging this pull request may close these issues.

None yet

2 participants