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

Add support of Block and CodeBlock #9

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

anobli
Copy link

@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
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

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.

None yet

3 participants