search

pygments / pygments.rb

💎 Ruby wrapper for Pygments syntax highlighter
MIT License
572 stars 141 forks source link

Updates pygments #127

Closed alindeman closed 10 years ago

alindeman commented 10 years ago

Includes Swift support among other updates; supercedes #122.

Notably, I could not reproduce the test failure mentioned in #122. Horray?

For review: @bkeepers @rick @tmm1 @arfon

arfon commented 10 years ago

If it's green, let's do it :+1:

bkeepers commented 10 years ago

:+1:

alindeman commented 10 years ago

For some reason, the TOML lexer isn't being picked up anymore. Need to dig into that before merge. I'll report back with what I find.

alindeman commented 10 years ago

Fixed with https://github.com/alindeman/pygments.rb/commit/d7066a95f72e7e1220e9e541d0bad9a3ef69bba6. I'm not sure how it ever worked without that, actually, since it's one of our custom lexers.

Ready for review again, I think: @tmm1, @tnm, @rodjek, @vmg

Afterward, I'll make a PR for linguist for the new lexers, including Swift.