New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix build with Markups 0.6. #63

Merged
merged 1 commit into from Feb 2, 2015

Conversation

Projects
None yet
3 participants
@loganchien
Contributor

loganchien commented Jan 31, 2015

The code was using internal API, thus cause the breakage after Markups 0.6.0 has been released.

This CL fixes the problem by using Markdown.set_output_format() and Markdown.registerExtensions() from the markdown module.

This CL also fixes a bug to initialize the super class properly.

@coveralls

This comment has been minimized.

coveralls commented Jan 31, 2015

Coverage Status

Changes Unknown when pulling 277efe4 on loganchien:fix-markups into * on mgaitan:master*.

1 similar comment
@coveralls

This comment has been minimized.

coveralls commented Jan 31, 2015

Coverage Status

Changes Unknown when pulling 277efe4 on loganchien:fix-markups into * on mgaitan:master*.

mgaitan added a commit that referenced this pull request Feb 2, 2015

Merge pull request #63 from loganchien/fix-markups
Fix build with Markups 0.6.

@mgaitan mgaitan merged commit 54d002a into mgaitan:master Feb 2, 2015

1 check passed

continuous-integration/travis-ci The Travis CI build passed
Details

@loganchien loganchien deleted the loganchien:fix-markups branch Feb 3, 2015

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