Skip to content

Unpin sphinx upper bound#9836

Closed
mxr wants to merge 1 commit into
pytest-dev:mainfrom
mxr:patch-1
Closed

Unpin sphinx upper bound#9836
mxr wants to merge 1 commit into
pytest-dev:mainfrom
mxr:patch-1

Conversation

@mxr
Copy link
Copy Markdown
Contributor

@mxr mxr commented Apr 4, 2022

CI fails with ImportError: cannot import name 'environmentfilter' from 'jinja2' (example).

Bumping Sphinx as described in sphinx-doc/sphinx#10291

CI fails with `ImportError: cannot import name 'environmentfilter' from 'jinja2'` ([example](https://github.com/pytest-dev/pytest/runs/5726444207)).

Bumping Sphinx as described in sphinx-doc/sphinx#10291
@mxr
Copy link
Copy Markdown
Contributor Author

mxr commented Apr 4, 2022

The tox docs step fails with

ERROR: InvocationError for command /home/runner/work/pytest/pytest/.tox/docs/bin/sphinx-build -W --keep-going -b html doc/en doc/en/_build/html -t changelog_towncrier_draft (exited with code 1)

I think this is from the below warning (since -W makes warnings into errors)

/home/runner/work/pytest/pytest/src/_pytest/pytester.py:docstring of _pytest.pytester.Pytester.TimeoutExpired:1: WARNING: duplicate object description of _pytest.pytester.Pytester.TimeoutExpired, other instance in reference/reference, use :noindex: for one of them

It seems there was some work to fix that in #8730 (comment) but no change was merged

Beyond that, CI will still be red because of psf/black#2964

@The-Compiler
Copy link
Copy Markdown
Member

This seems like a duplicate of #9248.

@pytest-dev pytest-dev deleted a comment from kshahaji166 May 28, 2022
bluetech added a commit to bluetech/pytest that referenced this pull request Aug 13, 2022
@bluetech
Copy link
Copy Markdown
Member

Indeed, we're handling this in #9248.

@bluetech bluetech closed this Aug 13, 2022
imgbot Bot pushed a commit to kkipngenokoech/pytest that referenced this pull request Apr 18, 2026
imgbot Bot pushed a commit to kkipngenokoech/pytest that referenced this pull request Apr 18, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants