Fix code-block in zoneinfo

[asottile]

Warning, treated as error:
/tmp/code/Doc/library/zoneinfo.rst:303:Error in "code-block" directive:
1 argument(s) required, 0 supplied.
.. code-block::
>>> a = ZoneInfo("Europe/Berlin")
>>> b = pickle.loads(europe_berlin_pkl)
>>> a is b
True

Automerge-Triggered-By: @pganssle

[asottile]

CC @ambv@pganssle this is breaking the 3.9.0b1 build for deadsnakes

[ambv]

Good catch, sorry for your trouble. Since the tag is already published, you'll have to cherrypick this for your build, is that okay?

[asottile]

Good catch, sorry for your trouble. Since the tag is already published, you'll have to cherrypick this for your build, is that okay?

yep! already applied it as a local patch which'll get removed for b1 🎉

[pganssle]

Thanks for the PR!

Does it not seem imprudent to enforce a more strict docs build on a redistribution than on upstream? Seems to me the feedback loop would be annoyingly long. Should we change our CI to catch these issues?

[miss-islington]

Thanks @asottile for the PR 🌮🎉.. I'm working now to backport this PR to: 3.9.
🐍🍒⛏🤖

[bedevere-bot]

GH-20204 is a backport of this pull request to the 3.9 branch.

[ambv]

The full release docs build is rather longish because it builds letter and A4 PDFs too with TeX etc. etc. So we only do that for RCs and stable releases. Making a minimal HTML build every time seems like something we want to have.

[pganssle]

@ambv Ah, thanks. I was thinking this was a build with -W or something (though come to think of it the backport for zoneinfo includes -W -n in the CI), in which case it would just be a matter of turning on that option, but It seems to me that -W wouldn't catch this, and -W -n, even if it would catch this, is not something we can turn on easily:

$ SPHINXOPTS="-W -a -j auto -n --keep-going" make html 1>/dev/null 2>error.log
$ wc -l error.log
6916 error.log

Thanks!