Downcase lexers before passing to Pygments #1615

Merged
merged 3 commits into from Oct 11, 2013

Projects

None yet

3 participants

@benbalter
Contributor

All available lexers for use with the highlight tag are lower case.e This is fine on case-insensitive file systems like OS X, but builds that work locally may fail on linux-based systems, which are case sensitive. (e.g., CSharp may work locally, while it fails on GItHub Pages).

Force lexers to be lower case before passing to Pygments.

@parkr
Member
parkr commented Oct 5, 2013

👍 @mattr-?

Thanks for the explanation, @benbalter :)

@parkr
Member
parkr commented Oct 6, 2013

Would you mind quickly adding a test for this?

@benbalter
Contributor

Done and done. As an added bonus, it actually works now. 😄

@parkr
Member
parkr commented Oct 7, 2013

Thank goodness for tests! :) Thanks, Ben!

@mattr-?

@mattr-
Member
mattr- commented Oct 11, 2013

ba da bing. ba da boom. Merged! 🚢 all the things! ❤️

@mattr- mattr- merged commit 0761769 into jekyll:master Oct 11, 2013

1 check passed

default The Travis CI build passed
Details
@mattr- mattr- added a commit that referenced this pull request Oct 11, 2013
@mattr- mattr- Update history to reflect merge of #1615 441b6d6
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment