Merge pull request #36735 from wgu-ram-chandra/issue-35271-unpin-markdown

chore: update markdown dependency to allow newer versions

Author: feanil

lms/djangoapps/course_wiki/plugins/markdownedx/mdx_mathjax.py

@@ -21,9 +21,9 @@ def handleMatch(self, m):
 
 
 class MathJaxExtension(markdown.Extension):
-    def extendMarkdown(self, md, md_globals):  # lint-amnesty, pylint: disable=arguments-differ, unused-argument
+    def extendMarkdown(self, md):  # lint-amnesty, pylint: disable=arguments-differ, unused-argument
         # Needs to come before escape matching because \ is pretty important in LaTeX
-        md.inlinePatterns.add('mathjax', MathJaxPattern(), '<escape')
+        md.inlinePatterns.register(MathJaxPattern(), 'mathjax', 180)
 
 
 def makeExtension(**kwargs):

lms/djangoapps/course_wiki/plugins/markdownedx/mdx_video.py

@@ -164,9 +164,9 @@ def add_inline(self, md, name, klass, re):  # pylint: disable=invalid-name
         pattern = klass(re)
         pattern.md = md
         pattern.ext = self
-        md.inlinePatterns.add(name, pattern, "<reference")
+        md.inlinePatterns.register(pattern, name, 170)
 
-    def extendMarkdown(self, md, md_globals):  # lint-amnesty, pylint: disable=arguments-differ, unused-argument
+    def extendMarkdown(self, md):  # lint-amnesty, pylint: disable=arguments-differ, unused-argument
         self.add_inline(md, 'bliptv', Bliptv,
                         r'([^(]|^)http://(\w+\.|)blip.tv/file/get/(?P<bliptvfile>\S+.flv)')
         self.add_inline(md, 'dailymotion', Dailymotion,

requirements/constraints.txt

@@ -64,12 +64,6 @@ edx-enterprise==6.0.2
 # https://github.com/openedx/edx-platform/issues/31616
 libsass==0.10.0
 
-# Date: 2018-12-14
-# markdown>=3.4.0 has failures due to internal refactorings which causes the tests to fail
-# pinning the version untill the issue gets resolved in the package itself
-# Issue for unpinning: https://github.com/openedx/edx-platform/issues/35271
-markdown<3.4.0
-
 # Date: 2024-07-16
 # We need to upgrade the version of elasticsearch to atleast 7.15 before we can upgrade to Numpy 2.0.0
 # Otherwise we see a failure while running the following command:

requirements/edx/base.txt

@@ -719,9 +719,8 @@ mako==1.3.10
     #   lti-consumer-xblock
     #   xblock
     #   xblock-utils
-markdown==3.3.7
+markdown==3.8
     # via
-    #   -c requirements/edx/../constraints.txt
     #   -r requirements/edx/kernel.in
     #   openedx-django-wiki
     #   staff-graded-xblock

requirements/edx/development.txt

@@ -1211,9 +1211,8 @@ mako==1.3.10
     #   lti-consumer-xblock
     #   xblock
     #   xblock-utils
-markdown==3.3.7
+markdown==3.8
     # via
-    #   -c requirements/edx/../constraints.txt
     #   -r requirements/edx/doc.txt
     #   -r requirements/edx/testing.txt
     #   openedx-django-wiki

requirements/edx/doc.txt

@@ -869,9 +869,8 @@ mako==1.3.10
     #   lti-consumer-xblock
     #   xblock
     #   xblock-utils
-markdown==3.3.7
+markdown==3.8
     # via
-    #   -c requirements/edx/../constraints.txt
     #   -r requirements/edx/base.txt
     #   openedx-django-wiki
     #   staff-graded-xblock

requirements/edx/testing.txt

@@ -925,9 +925,8 @@ mako==1.3.10
     #   lti-consumer-xblock
     #   xblock
     #   xblock-utils
-markdown==3.3.7
+markdown==3.8
     # via
-    #   -c requirements/edx/../constraints.txt
     #   -r requirements/edx/base.txt
     #   openedx-django-wiki
     #   staff-graded-xblock