-
Notifications
You must be signed in to change notification settings - Fork 21
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
Support latest markdown releases #7
base: master
Are you sure you want to change the base?
Conversation
The latest Markdown releases do no more require [md_globals](https://python-markdown.github.io/change_log/release-3.0/#md_globals-keyword-deprecated-from-extension-api) in `extendMarkdown`.
I came across the same issue and came to the same conclusion, so +1 with this change being correct. |
@oruelle - Any thoughts of merging this change and releasing an updated PyPi package? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Very required.
*edit: I read the source and I do have the most recent change; Upon inspection I have a different error: extendMarkdown() missing 1 required positional argument: 'md_globals'
Please ignore this approval.
@oruelle It looks like this change is straightforward to implement. Can you give it a look? |
@@ -85,7 +85,7 @@ def run(self, lines): | |||
class MermaidExtension(Extension): | |||
""" Add source code hilighting to markdown codeblocks. """ | |||
|
|||
def extendMarkdown(self, md, md_globals): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
md_globals=None for backwards compat, then there's no reason not to take this change.
Fix code to support the latest Python-Markdown releases, which do no more require md_globals in
extendMarkdown
.