Missing extensions for markdown when using Redcarpet

[jkowens]

This issue stems from an issue reported in the Sinatra repo: sinatra/sinatra/issues/1440.

The code following sums up the issue:

t = Tilt[:markdown] # => Tilt::Redcarpet2Template 
Tilt.default_mapping.extensions_for(t) # => ["markdown"] 

However notice:

Tilt.default_mapping.extensions_for("Tilt::RedcarpetTemplate") # => ["markdown", "mkd", "md"] 

I think the code to try to maintain Redcarpet1 compatibility is creating a problem. Can the Redcarpet1Template be eliminated at this point and we just have a RedcarpetTemplate class?

At a minimum I think the following change needs made, replace:

tilt/lib/tilt.rb

Line 127 in 24aff97

register_lazy :RedcarpetTemplate, 'tilt/redcarpet', 'markdown', 'mkd', 'md'

with:

register_lazy :Redcarpet1Template,    'tilt/redcarpet',    'markdown', 'mkd', 'md'
register_lazy :Redcarpet2Template,    'tilt/redcarpet',    'markdown', 'mkd', 'md'

[jkowens]

Ah didn't realize there was a PR to address this issue (#300). I still think consolidating down to just a single Tilt::RedcarpetTemplate class would be ideal.

[tommay]

It's not clear why that pull request (which is the same as your fix) didn't get merged or rejected three years ago.

[judofyr]

This should now be fixed in aea852b. Can you try latest Tilt-master and see it works?

[jkowens]

Thanks @judofyr! That did the trick.