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

The Examples formatting was using string concatenation, I needed more control and I needed to preprocess the docs to allow for changing the output at run time #327

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

Conversation

mark-ellul
Copy link

The examples were being formatted using string concatenation. I needed to have access at the template level.
So I added a new configuration property, render_examples_with_template, which allows the user to render the examples in the method description from a partial, once set, the new example partial will be used, or that partial can be overridden like the others templates.

NOTE: I couldn't get the Example partial to look like the string concatenation, so thats why its configurable.

…ich allows the user to render the examples in the method description from a partial, once set, the new example partial will be used.
@iNecas
Copy link
Member

iNecas commented Feb 11, 2015

Thanks. Looks promissing. What exactly were the issues for not being able to have a template to achieve the same results as with the string concatenation?

@mark-ellul
Copy link
Author

The Issue was that the string concatenation uses markup that I didn't know how it translates exactly to ERB, and it depends on which markup generator was being used.

Any ideas on that? Maybe a tag that outputs the markup as the String concat?

@mark-ellul mark-ellul changed the title The Examples formatting was using string concatenation, I needed more control The Examples formatting was using string concatenation, I needed more control and I needed to preprocess the docs to allow for changing the output at run time Feb 20, 2015
@ofedoren ofedoren force-pushed the master branch 2 times, most recently from 69442b9 to 8215e30 Compare July 25, 2021 19:41
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