-
-
Notifications
You must be signed in to change notification settings - Fork 32.1k
[3.7] bpo-47138: Fix documentation build by pinning Jinja version to 3.0.3 #32111
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
Conversation
See my comments on the bug issue. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This PR looks good, it only adds a jinja2 pin and doesn't modify the others.
The doc-html
preview downloadable at https://github.com/python/cpython/actions/runs/2041354404 looks good too.
Should we use this new bpo number? https://bugs.python.org/issue47138
I've updated the commit message with a new ticket number. |
@hugovk and @m-aciek, thanks! @m-aciek, perhaps an unnecessary warning but, before doing git pulls on your local repo, you probably will want to do a git reset on your 3.7 and 3.8 branches to remove the commits associated with this and your 3.8 PR as the commits get squashed when merging upstream and so will create an unnecessary merge in your local repo. That's why it's always best to use a unique branch name when creating a PR (like |
Yes, I'm sorry for the branch name, it probably made it harder to work with it locally. I used to use git pull --rebase, which should avoid the problem, but thank you for the hint nevertheless. And thank you for the cooperation on the fix! |
https://bugs.python.org/issue47138
https://bugs.python.org/issue45618#msg415979