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

fix: version deprecation warning from markdown #19

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

Conversation

michsien104
Copy link

@michsien104 michsien104 commented Jul 25, 2023

Closing #18
Fix of version imported from markdown which gives the deprecation warning.

Can you have a look here @radude for a review and get rid of this annoying deprecation warning ?

@ajschmidt8
Copy link

@radude, can we get this merged and released?

@chausner
Copy link

chausner commented Aug 16, 2023

Note that the attribute markdown.__version__ had been added in markdown 3.0.1. In version 3.0 and earlier, only markdown.version is present.

An alternative fix to the one proposed here could be to swap the try and except blocks, to default to the newer version and only try the old approach if the new one fails.

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.

None yet

4 participants