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

asciidoc related enhancements #54

Merged
merged 8 commits into from
Dec 12, 2013
Merged

asciidoc related enhancements #54

merged 8 commits into from
Dec 12, 2013

Conversation

Vad1mo
Copy link
Member

@Vad1mo Vad1mo commented Nov 8, 2013

Hello,

I made some changes to your code. You might find them as useful as I do.

  • jbake now does not treat all types other then page as post. The reason ist that normally it is fine to have the same template for all posts. But for pages you might define a different type and thus a different template.
  • Set asciidoc renderer to unsafe and the base dir to jbake root.

Cheers,
Vadim

avoids having problems in feed and archive with dates. Allows us to
define another templates for each page type.
because WE are generating our sites. 
2. asciidoc set base dir to be jbake root, now it is easy to
include::content/* in our asciidoc sites.
@coveralls
Copy link

Coverage Status

Coverage increased (+0.08%) when pulling f8fccd7 on Vad1mo:master into 328acce on jonbullock:master.

@jonbullock
Copy link
Member

Thanks for the contribution! I may look at separating out the changes, so the Asciidoctor safe mode change is included in v2.2.1 and the change to the convention that everything is a post by default to a later release, might be best to re-think the situation completely.

before the latest overwrote the previous. The default.property
asciidoctor.options=source-highlighter=prettify if still overwritten
when custom property is defined.
Because we don't know the type we guess them by their content. not
perfect but ok for boolean,int and string.
@ghost ghost assigned jonbullock Dec 9, 2013
jonbullock added a commit that referenced this pull request Dec 12, 2013
Reverted change in convention re: posts & pages for time being. Also set
Asciidoctor to UNSAFE mode in code.
@jonbullock jonbullock merged commit 968bb7e into jbake-org:master Dec 12, 2013
@jonbullock
Copy link
Member

Thanks again for the contribution. Hope you don't mind but I've hard coded the Asciidoctor safe mode to unsafe, because I couldn't think of a use case when you'd want to change this value, but if you have a scenario please let me know. I'll raise a separate issue to change the convention.

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