This is an archive of the discontinued LLVM Phabricator instance.

Differential D95069

Install future before running sphinx
ClosedPublic

Authored by thopre on Jan 20 2021, 10:22 AM.

Details

Reviewers
tnfchris
danilaml
cmatthews
Commits
rLNTf48d431f4461: Install future before running sphinx
Summary

Fix Sphinx warning about lack of module named 'future' by making the
installation of future a dependency of sphinx in tox.ini

Diff Detail

Repository
rLNT LNT

Event Timeline

thopre requested review of this revision.Jan 20 2021, 10:22 AM
thopre created this revision.
Harbormaster completed remote builds in B85927: Diff 317929.Jan 20 2021, 10:22 AM
danilaml accepted this revision.Jan 20 2021, 10:32 AM

LGTM 👍

This revision is now accepted and ready to land.Jan 20 2021, 10:32 AM
Closed by commit rLNTf48d431f4461: Install future before running sphinx (authored by thopre). · Explain WhyJan 20 2021, 10:43 AM
This revision was automatically updated to reflect the committed changes.
thopre added a commit: rLNTf48d431f4461: Install future before running sphinx.

Revision Contents

PathSize
1 line

Diff 317940

tox.ini

Show All 32 Linescommands =
# Lets keep the clean files clean. # Lets keep the clean files clean.
flake8 --statistics --exclude=./lnt/external/ ./lnt/ ./tests/ ./setup.py ./deployment/ flake8 --statistics --exclude=./lnt/external/ ./lnt/ ./tests/ ./setup.py ./deployment/
[testenv:docs] [testenv:docs]
skip_install = true skip_install = true
deps = deps =
sphinx<2.0.0 sphinx<2.0.0
sphinx_bootstrap_theme sphinx_bootstrap_theme
future
commands = commands =
make -C {toxinidir}/docs/ html make -C {toxinidir}/docs/ html