Remove literal lang name from class, fix #2284 #2292

Merged
merged 2 commits into from May 3, 2014

Conversation

Projects
None yet
3 participants
@vtols
Contributor

vtols commented May 3, 2014

If code language is C classes for comment and language are the same

Remove literal lang name from class, fix #2284
If code language is C classes for comment and language are the same
@vtols

This comment has been minimized.

Show comment
Hide comment
@vtols

vtols May 3, 2014

Contributor

So, as can I see, idea to remove literal language name from class (this comment) isn't good, because build fails 😕. Any other ideas?

Contributor

vtols commented May 3, 2014

So, as can I see, idea to remove literal language name from class (this comment) isn't good, because build fails 😕. Any other ideas?

Update tests
Remove language literal from class in test/test_redcarpet.rb
@vtols

This comment has been minimized.

Show comment
Hide comment
@vtols

vtols May 3, 2014

Contributor

I'm updated tests, now all is OK with them, but there is some bug concerning Travis, maybe we need to fix it and restart build

Contributor

vtols commented May 3, 2014

I'm updated tests, now all is OK with them, but there is some bug concerning Travis, maybe we need to fix it and restart build

@parkr parkr merged commit 9cd648a into jekyll:master May 3, 2014

1 check failed

continuous-integration/travis-ci The Travis CI build could not complete due to an error
Details

parkr added a commit that referenced this pull request May 3, 2014

@jekyllbot jekyllbot locked and limited conversation to collaborators Feb 27, 2017

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.