Skip to content

fix errors in new sphinx version#1355

Closed
kain88-de wants to merge 1 commit intoMDAnalysis:developfrom
kain88-de:doc-build-fix
Closed

fix errors in new sphinx version#1355
kain88-de wants to merge 1 commit intoMDAnalysis:developfrom
kain88-de:doc-build-fix

Conversation

@kain88-de
Copy link
Copy Markdown
Member

Changes made in this Pull Request:

  • cite reference Seyler2015b to fix sphinx build.

PR Checklist

- [ ] Tests?

  • Docs?
    - [ ] CHANGELOG updated?
    - [ ] Issue raised/referenced?

@kain88-de
Copy link
Copy Markdown
Member Author

Turns out there are more reference issues. I assume this is due to a sphinx update. I managed to fix some but I don't know why there is an error in encore.rst about missing dereferences.

@kain88-de
Copy link
Copy Markdown
Member Author

See sphinx-doc/sphinx#3565

@kain88-de kain88-de changed the title cite reference Seyler2015b to fix sphinx build. fix errors in new sphinx version May 19, 2017
@orbeckst
Copy link
Copy Markdown
Member

This is fixed in PR #1126 – I think we will be able to close this PR after merging #1126

@kain88-de
Copy link
Copy Markdown
Member Author

@orbeckst can this be closed then? Do you know about the current error I'm getting?

@orbeckst
Copy link
Copy Markdown
Member

I didn't know about any false warnings in sphinx. I just got the failures due to citations and made sure that everything is referenced. In particular, I remember looking at the encore docs and wondering why it didn't work. I think I worked around it in 8cd7603 by just renaming the reference... weird. I made the same changes in references that you have here.

PR #1126 builds fine with sphinx 1.6.1 and warnings as errors so I am pretty sure this PR can be closed.

I will do so and we can re-open if necessary.

@orbeckst orbeckst closed this May 19, 2017
@kain88-de kain88-de deleted the doc-build-fix branch May 19, 2017 21:39
@kain88-de kain88-de restored the doc-build-fix branch May 30, 2017 18:55
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.

2 participants