sphinx-contrib / video

A sphinx plugin that enables embedding of HTML5 videos
https://sphinxcontrib-video.readthedocs.io
Apache License 2.0
43 stars 29 forks source link

0.2.0: pytest warnings #31

Closed kloczek closed 1 year ago

kloczek commented 1 year ago

Looks like pytest shows some warnings with sphinx 6.2.1

+ PYTHONPATH=/home/tkloczko/rpmbuild/BUILDROOT/python-sphinxcontrib-video-0.2.0-2.fc35.x86_64/usr/lib64/python3.8/site-packages:/home/tkloczko/rpmbuild/BUILDROOT/python-sphinxcontrib-video-0.2.0-2.fc35.x86_64/usr/lib/python3.8/site-packages
+ /usr/bin/pytest -ra -m 'not network'
==================================================================================== test session starts ====================================================================================
platform linux -- Python 3.8.17, pytest-7.4.0, pluggy-1.0.0
rootdir: /home/tkloczko/rpmbuild/BUILD/video-0.2.0
plugins: datadir-1.4.1, regressions-2.4.2
collected 8 items

tests/test_build.py ........                                                                                                                                                          [100%]

===================================================================================== warnings summary ======================================================================================
tests/test_build.py::test_video_latex
tests/test_build.py::test_video_latex
  /usr/lib/python3.8/site-packages/sphinxcontrib/htmlhelp/__init__.py:26: RemovedInSphinx80Warning: The alias 'sphinx.util.progress_message' is deprecated, use 'sphinx.http_date.epoch_to_rfc1123' instead. Check CHANGES for Sphinx API modifications.
    from sphinx.util import progress_message

-- Docs: https://docs.pytest.org/en/stable/how-to/capture-warnings.html
=============================================================================== 8 passed, 2 warnings in 1.30s ===============================================================================
kloczek commented 1 year ago

BTW looks like sphinxcontrib/__init__.py file can be removed (it uses as well deprecated pkg_resources module which needs to be used only with old style namespaces).

kloczek commented 1 year ago

Eh .. sorry I did not realised that that warning comes from external module 😞 Nevertheless part about remove sphinxcontrib/__init__.py still is valid.

12rambau commented 1 year ago

agreed that was already requested from me here: https://github.com/sphinx-contrib/github-administration/issues/38, I should have apply it to myself....