search

PyWavelets / pywt

PyWavelets - Wavelet Transforms in Python
http://pywavelets.readthedocs.org
MIT License
1.97k stars 460 forks source link

DEP: Remove `docutils` upper-pin, bump `sphinx` #739

Closed agriyakhetarpal closed 2 months ago

agriyakhetarpal commented 2 months ago

Description

This PR is a follow-up from #737 to bump some documentation-related dependencies in util/readthedocs/requirements.txt, as requested in https://github.com/PyWavelets/pywt/pull/737#pullrequestreview-2033172515.

Changes made

  1. Added a minimum version requirement for sphinx – set to>=7`
  2. Removed the constraint from docutils – it was the one that was blocking this change.

Additional context

I checked many of the pages personally and there seem to be no problems with the build. The documentation does not display any warnings and the build with -W passes locally. All of our extensions and other listed dependencies support Sphinx v7 as well.

agriyakhetarpal commented 2 months ago

Logs from Read the Docs dashboard: https://beta.readthedocs.org/projects/pywavelets/builds/24244868/

The build passes here as well, no warnings emitted.