Skip to content
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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Ircama
Copy link

@Ircama Ircama commented Aug 8, 2022

Fix code to support the latest Python-Markdown releases, which do no more require md_globals in extendMarkdown.

@rayalan
Copy link

rayalan commented Aug 17, 2022

I came across the same issue and came to the same conclusion, so +1 with this change being correct.

@rayalan
Copy link

rayalan commented Sep 13, 2022

@oruelle - Any thoughts of merging this change and releasing an updated PyPi package?

Copy link

@Strangemother Strangemother left a 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.

@Pebaz
Copy link

Pebaz commented Jul 18, 2023

@oruelle It looks like this change is straightforward to implement. Can you give it a look?
I'm running into the extendMarkdown() missing 1 required positional argument: 'md_globals' as well.

@EricPostMaster
Copy link

@oruelle, @Pebaz - Is this going to be merged soon? I'd like to use mermaid, and this is blocking me.

@@ -85,7 +85,7 @@ def run(self, lines):
class MermaidExtension(Extension):
""" Add source code hilighting to markdown codeblocks. """

def extendMarkdown(self, md, md_globals):
Copy link

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants