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

Fix syntax in RST tables #968

Merged
merged 1 commit into from May 1, 2018
Merged

Fix syntax in RST tables #968

merged 1 commit into from May 1, 2018

Conversation

mwtoews
Copy link
Member

@mwtoews mwtoews commented May 1, 2018

Fixes LaTeX issue in #967 and allows PDF to be built on older sphinx versions (like mine)

@kbevers
Copy link
Member

kbevers commented May 1, 2018

Nice catch. I wonder why Sphinx didn't complain about that here...?

@kbevers kbevers merged commit 2f467d5 into OSGeo:master May 1, 2018
@mwtoews
Copy link
Member Author

mwtoews commented May 1, 2018

Yeah, I'd like to know a better way to parse .rst documents to raise possible errors or warnings. I only found these, and similar (410a515) by inspecting the latex log output to see which document it was parsing, then looking a the .rst document to manually inspect for oddities. There should be a better way to log these somehow...

@kbevers
Copy link
Member

kbevers commented May 1, 2018

Apparently there's rstcheck which seems to do the trick. More or less at least. I'll play around with it a bit. I might make sense to add as a travis check.

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