Skip to content

Commit

Permalink
Added Redcarpet for MD conversion
Browse files Browse the repository at this point in the history
  • Loading branch information
MattHall committed Apr 20, 2011
1 parent dce4ccc commit cf779b2
Show file tree
Hide file tree
Showing 8 changed files with 60 additions and 1 deletion.
4 changes: 4 additions & 0 deletions bin/jekyll
Expand Up @@ -82,6 +82,10 @@ opts = OptionParser.new do |opts|
opts.on("--rdiscount", "Use rdiscount gem for Markdown") do
options['markdown'] = 'rdiscount'
end

opts.on("--redcarpet", "Use redcarpet gem for Markdown") do
options['markdown'] = 'redcarpet'
end

opts.on("--kramdown", "Use kramdown gem for Markdown") do
options['markdown'] = 'kramdown'
Expand Down
7 changes: 7 additions & 0 deletions features/site_configuration.feature
Expand Up @@ -55,6 +55,13 @@ Feature: Site configuration
Then the _site directory should exist
And I should see "<a href="http://google.com">Google</a>" in "_site/index.html"

Scenario: Use Redcarpet for markup
Given I have an "index.markdown" page that contains "[Google](http://google.com)"
And I have a configuration file with "markdown" set to "redcarpet"
When I run jekyll
Then the _site directory should exist
And I should see "<a href="http://google.com">Google</a>" in "_site/index.html"

Scenario: Use Maruku for markup
Given I have an "index.markdown" page that contains "[Google](http://google.com)"
And I have a configuration file with "markdown" set to "maruku"
Expand Down
3 changes: 2 additions & 1 deletion jekyll.gemspec
Expand Up @@ -36,7 +36,8 @@ Gem::Specification.new do |s|
s.add_development_dependency('cucumber', ">= 0.10.0")
s.add_development_dependency('RedCloth', ">= 4.2.1")
s.add_development_dependency('rdiscount', ">= 1.6.5")

s.add_development_dependency('redcarpet', ">= 1.9.0")

# = MANIFEST =
s.files = %w[
History.txt
Expand Down
3 changes: 3 additions & 0 deletions lib/jekyll.rb
Expand Up @@ -76,6 +76,9 @@ module Jekyll
'rdiscount' => {
'extensions' => []
},
'redcarpet' => {
'extensions' => []
},
'kramdown' => {
'auto_ids' => true,
'footnote_nr' => 1,
Expand Down
11 changes: 11 additions & 0 deletions lib/jekyll/converters/markdown.rb
Expand Up @@ -10,6 +10,15 @@ def setup
return if @setup
# Set the Markdown interpreter (and Maruku self.config, if necessary)
case @config['markdown']
when 'redcarpet'
begin
require 'redcarpet'
@redcarpet_extensions = @config['redcarpet']['extensions'].map { |e| e.to_sym }
rescue LoadError
STDERR.puts 'You are missing a library required for Markdown. Please run:'
STDERR.puts ' $ [sudo] gem install redcarpet'
raise FatalException.new("Missing dependency: redcarpet")
end
when 'kramdown'
begin
require 'kramdown'
Expand Down Expand Up @@ -77,6 +86,8 @@ def output_ext(ext)
def convert(content)
setup
case @config['markdown']
when 'redcarpet'
Redcarpet.new(content, *@redcarpet_extensions).to_html
when 'kramdown'
# Check for use of coderay
if @config['kramdown']['use_coderay']
Expand Down
1 change: 1 addition & 0 deletions test/helper.rb
Expand Up @@ -6,6 +6,7 @@
require 'RedCloth'
require 'rdiscount'
require 'kramdown'
require 'redcarpet'

require 'test/unit'
require 'redgreen'
Expand Down
21 changes: 21 additions & 0 deletions test/test_redcarpet.rb
@@ -0,0 +1,21 @@
require File.dirname(__FILE__) + '/helper'

class TestRedcarpet < Test::Unit::TestCase
context "redcarpet" do
setup do
config = {
'redcarpet' => { 'extensions' => ['smart'] },
'markdown' => 'redcarpet'
}
@markdown = MarkdownConverter.new config
end

should "pass redcarpet options" do
assert_equal "<h1>Some Header</h1>", @markdown.convert('# Some Header #').strip
end

should "pass redcarpet extensions" do
assert_equal "<p>&ldquo;smart&rdquo;</p>", @markdown.convert('"smart"').strip
end
end
end
11 changes: 11 additions & 0 deletions test/test_tags.rb
Expand Up @@ -123,5 +123,16 @@ def fill_post(code, override = {})
assert_match %r{<em>FINISH HIM</em>}, @result
end
end

context "using Redcarpet" do
setup do
create_post(@content, 'markdown' => 'redcarpet')
end

should "parse correctly" do
assert_match %r{<em>FIGHT!</em>}, @result
assert_match %r{<em>FINISH HIM</em>}, @result
end
end
end
end

0 comments on commit cf779b2

Please sign in to comment.