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

Outdated docutils dependendancy #1570

Closed
sjfke opened this issue Jul 11, 2024 · 4 comments · Fixed by #1576
Closed

Outdated docutils dependendancy #1570

sjfke opened this issue Jul 11, 2024 · 4 comments · Fixed by #1576
Labels
Bug A bug Needed: replication Bug replication is required

Comments

@sjfke
Copy link

sjfke commented Jul 11, 2024

Problem

GitHub dependabot demands docutils==0.21.2 but 'sphinx' and 'sphinx-rtd-theme' require outdated versions.
Using docutils<=0.21.2 does not make dependabot happy.

Give details on your issue.

Can you update the docutils dependency to unravel this dependency issue?

Reproducible Project

See https://github.com/sjfke/nonbleedingedge/ (https://nonbleedingedge.com/)

$ cat requirements.txt
alabaster==0.7.16
Babel==2.15.0
certifi==2024.7.4
charset-normalizer==3.3.2
click==8.1.7
colorama==0.4.6
docutils==0.21.2
idna==3.7
imagesize==1.4.1
importlib_metadata==8.0.0
Jinja2==3.1.4
MarkupSafe==2.1.5
packaging==24.1
Pygments==2.18.0
pyparsing==3.1.2
pytz==2024.1
requests==2.32.3
setuptools==70.3.0
snowballstemmer==2.2.0
Sphinx==7.3.7
sphinx-intl==2.2.0
sphinx-rtd-theme==2.0.0
sphinxcontrib-applehelp==1.0.8
sphinxcontrib-devhelp==1.0.6
sphinxcontrib-htmlhelp==2.0.5
sphinxcontrib-jquery==4.1
sphinxcontrib-jsmath==1.0.1
sphinxcontrib-qthelp==1.0.7
sphinxcontrib-serializinghtml==1.1.10
sphinxemoji==0.3.1
urllib3==2.2.2
zipp==3.19.2

$ pip install -r requirements.txt --upgrade

Error Logs/Results

The conflict is caused by:
The user requested docutils==0.21.2
sphinx 7.3.7 depends on docutils<0.22 and >=0.18.1
sphinx-rtd-theme 2.0.0 depends on docutils<0.21
The user requested docutils==0.21.2
sphinx 7.3.7 depends on docutils<0.22 and >=0.18.1
sphinx-rtd-theme 2.0.0 depends on docutils<0.21

Expected Results

No errors

Environment Info

  • Python Version: Python 3.12.4 (From Microsoft Store)
  • Sphinx Version: 7.3.7
  • RTD Theme Version: 2.0.0
@sjfke sjfke added Bug A bug Needed: replication Bug replication is required labels Jul 11, 2024
@sjfke
Copy link
Author

sjfke commented Aug 5, 2024

Hi Guys, I appreciate it is Summer Vacation time and the Olympics are on, but could you please acknowledge.
Who knows, you might be fixing upstream security issues by looking at this?

@humitos
Copy link
Member

humitos commented Aug 7, 2024

This is going to happen in 3.0, and it's tracked by #1576

@humitos
Copy link
Member

humitos commented Aug 20, 2024

We released a new version 3.0.0rc1 that adds support for latest docutils versions. Please, give it a try and open an issue for any problem or feedback you want to share with us.

@sjfke
Copy link
Author

sjfke commented Aug 22, 2024

The following works but also updated sphinx and a few others...
FWIW: I am using PyCharm CE to determine what needs updating

$ cat requirements.txt
alabaster==1.0.0
Babel==2.16.0
certifi==2024.7.4
charset-normalizer==3.3.2
click==8.1.7
colorama==0.4.6
docutils==0.21.2
idna==3.7
imagesize==1.4.1
importlib_metadata==8.4.0
Jinja2==3.1.4
MarkupSafe==2.1.5
packaging==24.1
Pygments==2.18.0
pyparsing==3.1.2
pytz==2024.1
requests==2.32.3
setuptools==73.0.1
snowballstemmer==2.2.0
Sphinx==8.0.2
sphinx-intl==2.2.0
sphinx-rtd-theme==3.0.0rc1
sphinxcontrib-applehelp==2.0.0
sphinxcontrib-devhelp==2.0.0
sphinxcontrib-htmlhelp==2.1.0
sphinxcontrib-jquery==4.1
sphinxcontrib-jsmath==1.0.1
sphinxcontrib-qthelp==2.0.0
sphinxcontrib-serializinghtml==1.1.10
sphinxemoji==0.3.1
urllib3==2.2.2
zipp==3.20.0

$ pip install -r requirements.txt --upgrade

Website builds with no errors and looks fine (don't have an automated testing setup)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug A bug Needed: replication Bug replication is required
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants