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

Switch to Pegdown Markdown processor from MarkdownJ #57

Closed
wants to merge 11 commits into from
Closed

Switch to Pegdown Markdown processor from MarkdownJ #57

wants to merge 11 commits into from

Conversation

ksclarke
Copy link
Contributor

Hi, I'm wondering if you'd consider a switch to the Pegdown Markdown processor. It has support for some very useful Markdown extensions (mostly those from GitHub and PHP). MarkdownJ has plans to support extensions, but doesn't currently. I've switched to Pegdown in this pull request and created some tests for the extra features (in the process, splitting the Markdown parser tests out into their own file). Thanks for your consideration and thanks for creating jbake.

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.
@s17t
Copy link
Contributor

s17t commented Nov 25, 2013

+1 for a modern MD version.

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.
@jonbullock
Copy link
Member

Yeah I'd definitely consider it, as I've already had users report issues with the current library.

Thanks very much for your contribution and support. :-)

@ghost ghost assigned jonbullock Dec 9, 2013
@kongdata
Copy link

+1 to see Pegdown as the markdown processor as well (I've used it myself as a lib and its very nice). Been able to format tables in markdown is the feature I'm after from jbake.

Reverted change in convention re: posts & pages for time being. Also set
Asciidoctor to UNSAFE mode in code.
@ksclarke ksclarke closed this Dec 15, 2013
@ksclarke ksclarke deleted the pegdown branch December 15, 2013 23:39
@ksclarke ksclarke reopened this Dec 15, 2013
@ksclarke ksclarke closed this Dec 15, 2013
@ksclarke
Copy link
Contributor Author

Bah, I wanted to put all my changes in a single commit but I've botched the process so I'll have to do some cleanup and submit another pull request.

@ksclarke ksclarke reopened this Dec 16, 2013
@ksclarke ksclarke closed this Dec 16, 2013
@ksclarke
Copy link
Contributor Author

Okay, since I've made a mess of this PR, I've cleaned it up in a new one: https://github.com/jonbullock/JBake/pull/63

Sorry for the botched clean-up of this one.

@jonbullock
Copy link
Member

No problem, I'll take a look at the new PR later on this week.

jonbullock pushed a commit to jonbullock/jbake that referenced this pull request Feb 11, 2022
…ry-usage-without-regression

Use the worker API for Gradle 5.6 and above
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.

5 participants