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

bpo-43774: Doc job of Azure Pipelines uses Doc/requirements.txt #25296

Merged
merged 1 commit into from Apr 9, 2021

Conversation

@vstinner
Copy link
Member

@vstinner vstinner commented Apr 8, 2021

Don't hardcode the Sphinx version but use Doc/requirements.txt.

https://bugs.python.org/issue43774

Automerge-Triggered-By: GH:tiran

Don't hardcode the Sphinx version but use Doc/requirements.txt.
@vstinner
Copy link
Member Author

@vstinner vstinner commented Apr 9, 2021

Oh, Azure Pipelines skipped the Doc PR job. I modified the workflow temporarily to force running the Docs PR job.

@vstinner vstinner force-pushed the vstinner:azure_reqs branch from 1553048 to 5789ff6 Apr 9, 2021
@vstinner
Copy link
Member Author

@vstinner vstinner commented Apr 9, 2021

Ok, it works as expected:

python -m pip install -r requirements.txt
...
Collecting sphinx==3.2.1
  Downloading Sphinx-3.2.1-py3-none-any.whl (2.9 MB)

With this version, no warning is emitted.

@vstinner vstinner force-pushed the vstinner:azure_reqs branch from 5789ff6 to 7d13f36 Apr 9, 2021
@methane
methane approved these changes Apr 9, 2021
@miss-islington miss-islington merged commit f7be26a into python:master Apr 9, 2021
16 checks passed
16 checks passed
Check for source changes
Details
Check for source changes
Details
Check if generated files are up to date
Details
Check if generated files are up to date
Details
Windows (x86)
Details
Windows (x86)
Details
Windows (x64)
Details
Windows (x64)
Details
macOS
Details
macOS
Details
Ubuntu
Details
Ubuntu
Details
Azure Pipelines PR #20210409.4 succeeded
Details
Travis CI - Pull Request Build Passed
Details
bedevere/issue-number Issue number 43774 found
Details
bedevere/news "skip news" label found
@vstinner vstinner deleted the vstinner:azure_reqs branch Apr 9, 2021
@stefanrad25
Copy link

@stefanrad25 stefanrad25 commented Apr 9, 2021

@vstinner
Copy link
Member Author

@vstinner vstinner commented Apr 9, 2021

@stefanrad25: I don't understand your remark.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
7 participants