Skip to content

Add support of Block and CodeBlock#9

Open
anobli wants to merge 2 commits into
michaelstepner:mainfrom
BayLibre:master
Open

Add support of Block and CodeBlock#9
anobli wants to merge 2 commits into
michaelstepner:mainfrom
BayLibre:master

Conversation

@anobli
Copy link
Copy Markdown

@anobli anobli commented Sep 12, 2019

Currently, the variable substitution is only made for text.
This updates pandoc-mustache to also substitute variables in Block
and CodeBlock.

Signed-off-by: Alexandre Bailon abailon@baylibre.com

Currently, the variable substitution is only made for text.
This updates pandoc-mustache to also substitute variables from Block
and CodeBlock.

Signed-off-by: Alexandre Bailon <abailon@baylibre.com>
@michaelstepner
Copy link
Copy Markdown
Owner

@anobli: This is a good idea! Thanks for submitting this PR.

Would you be up for adding two tests verifying that pandoc-mustache successfully substitutes variables in Block and CodeBlock environments? You could add them as one or two Python scripts in tests/, using the existing tests as examples.

@anobli anobli force-pushed the master branch 2 times, most recently from f28650e to 72cd99d Compare September 16, 2019 14:57
This adds a new test to check that variable substitution is working
correctly for Code or Codeblock.

Note:
The test is mostly a copy past of test_mustach_in_abstract.

Signed-off-by: Alexandre Bailon <abailon@baylibre.com>
@makohoek
Copy link
Copy Markdown

Hi @michaelstepner ,

It seems that @anobli implemented the tests you've requested in 72cd99d

Is there anything preventing from merging this PR?

I would like to get rid of our internal fork we use at BayLibre.

We have been using both patches for quite some time already.

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.

3 participants