chore: update markdown dependency to allow newer versions#36735
chore: update markdown dependency to allow newer versions#36735feanil merged 5 commits intoopenedx:masterfrom
Conversation
|
Thanks for the pull request, @wgu-ram-chandra! This repository is currently maintained by Once you've gone through the following steps feel free to tag them in a comment and let them know that your changes are ready for engineering review. 🔘 Get product approvalIf you haven't already, check this list to see if your contribution needs to go through the product review process.
🔘 Provide contextTo help your reviewers and other members of the community understand the purpose and larger context of your changes, feel free to add as much of the following information to the PR description as you can:
🔘 Get a green buildIf one or more checks are failing, continue working on your changes until this is no longer the case and your build turns green. DetailsWhere can I find more information?If you'd like to get more details on all aspects of the review process for open source pull requests (OSPRs), check out the following resources: When can I expect my changes to be merged?Our goal is to get community contributions seen and reviewed as efficiently as possible. However, the amount of time that it takes to review and merge a PR can vary significantly based on factors such as:
💡 As a result it may take up to several weeks or months to complete a review and merge your PR. |
|
2U Release Notice: This PR has been deployed to the edX staging environment in preparation for a release to production. |
|
2U Release Notice: This PR has been deployed to the edX production environment. |
|
2U Release Notice: This PR has been deployed to the edX staging environment in preparation for a release to production. |
|
2U Release Notice: This PR has been deployed to the edX production environment. |
|
2U Release Notice: This PR has been deployed to the edX staging environment in preparation for a release to production. |
|
2U Release Notice: This PR has been deployed to the edX production environment. |
Fixes: #35271
Description
This PR addresses a compatibility issue with the
markdownlibrary inedx-platform. Starting from version3.4.0,markdownintroduced several breaking changes, including:md_globalsargument from extension hooks.add()with.register()for processor registrationregister()method's argument signature: now follows the format(<processor_instance>, <name>, <priority:int>)These changes broke compatibility with dependencies such as
openedx-django-wiki, which was still using the older Markdown API.What’s Been Done
AnchorTagExtensionclass inopenedx-django-wikiwas not compatible withmarkdown>=3.4.0.openedx-django-wikirepo: openedx/django-wiki#3033.1.1of theopenedx-django-wikipackage.markdown<3.4.0constraint and allowsmarkdown==3.8(latest at time of writing), ensuring compatibility across both repos.References
openedx-django-wiki: refactor: update markdown to 3.8 and adapt extension code for compatibility django-wiki#303