Skip to content
This repository has been archived by the owner on Jan 25, 2019. It is now read-only.

Issue 144 rst source path take2 #161

Merged
merged 3 commits into from
Jul 18, 2017

Conversation

matt-garman
Copy link
Contributor

Another PR broke the previous attempt at fixing issue #144. Looks like now we are now passing page.meta to the render() function. wok/page.py line 120 has comment, "the page.meta might contain renderer options". So I added a new metadata field, "source_file". This value is the input source file name.

For the reStructuredText render, page_meta['source_path'] will be passed to docutils.core.publish_parts().

This allows the source filename to be printed if the renderer encounters an error.

The reStructuredText render will pass use this metadata if availbe,
and pass through to the source_path parameter in publich_parts.
This is to fix issue mythmon#144.
docutils.core.publish_parts() source_path parameter.  This is to fix
issue mythmon#144.
@edunham edunham merged commit c5b6100 into mythmon:master Jul 18, 2017
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.

2 participants