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

Complete edx-sphinx-theme DEPR in edx org #339

Open
15 of 18 tasks
robrap opened this issue Jun 21, 2023 · 2 comments
Open
15 of 18 tasks

Complete edx-sphinx-theme DEPR in edx org #339

robrap opened this issue Jun 21, 2023 · 2 comments
Assignees

Comments

@robrap
Copy link
Contributor

robrap commented Jun 21, 2023

AC:

  • Usage of edx-sphinx-theme in the edx org has been removed.

Additional Notes:

Tasks

@huniafatima-arbi
Copy link
Member

We are waiting on the information to be added in license and author section which is to be updated as the edx-sphinx-theme is being replaced by sphinx-book-theme.
Slack thread for more information.
Moving it to blocked until the issue is resolved.

@huniafatima-arbi huniafatima-arbi moved this from Author Team Review to Blocked in Arbi-BOM Oct 21, 2024
@MichaelRoytman
Copy link
Member

@huniafatima-arbi I hope you're doing well. I think that the upgrade of sphinx may have broken the requirements in the repository, as there is now a conflict between doc8, readme-renderer, and sphinx on the version of docutils. This causing the Python requirements upgrade action to fail.

Here is the error log.

pip._vendor.resolvelib.resolvers.ResolutionImpossible: [RequirementInformation(requirement=SpecifierRequirement('docutils<=0.21.2,>=0.19'), parent=LinkCandidate('https://files.pythonhosted.org/packages/0c/f1/6ffd5d76578e98a8f21ae7216b88a7212c778f665f1a8f4f8ce6f9605da4/doc8-1.1.2-py3-none-any.whl (from https://pypi.org/simple/doc8/) (requires-python:>=3.8)')), RequirementInformation(requirement=SpecifierRequirement('docutils>=0.21.2'), parent=LinkCandidate('https://files.pythonhosted.org/packages/e1/67/921ec3024056483db83953ae8e48079ad62b92db7880013ca77632921dd0/readme_renderer-44.0-py3-none-any.whl (from https://pypi.org/simple/readme-renderer/) (requires-python:>=3.9)')), RequirementInformation(requirement=SpecifierRequirement('docutils<0.20,>=0.14'), parent=LinkCandidate('https://files.pythonhosted.org/packages/67/a7/01dd6fd9653c056258d65032aa09a615b5d7b07dd840845a9f41a8860fbc/sphinx-5.3.0-py3-none-any.whl (from https://pypi.org/simple/sphinx/) (requires-python:>=3.6)'))]
  • doc8 requires docutils<=0.21.2,>=0.19
  • readme-renderer requires docutils>=0.21.2
  • sphinx requires docutils<0.20,>=0.14

The earliest version of sphinx that supports docutils>0.21.0 is 7.3.0, but that causes conflicts with the sphinx-book-theme you added, because that requires sphinx<7.

I'm unable to find a combination of versions that allow us to successfully install requirements. Can you help me understand how to correct this issue while maintaining the changes you made? Thank you.

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

No branches or pull requests

3 participants