Repository navigation
gh-152633: Fix outdated default compresslevel in gzip docs - #152634
Jaden-Varkey wants to merge 2 commits into
Conversation
The default compression level for gzip.open(), the GzipFile constructor and gzip.compress() was lowered from 9 to 6 (it now uses _COMPRESS_LEVEL_TRADEOFF), and the signatures and versionchanged notes were updated accordingly. A couple of docstrings and one paragraph in the library docs still said the default was 9, which was misleading. Update them to match the actual default of 6.
|
All commit authors signed the Contributor License Agreement. |
|
Most changes to Python require a NEWS entry. Add one using the blurb_it web app or the blurb command-line tool. If this change has little impact on Python users, wait for a maintainer to apply the |
Documentation build overview
|
|
This is a documentation-only change (two docstrings plus one paragraph in gzip.rst), so it shouldn't need a Misc/NEWS entry; could a skip news label be added? Appreciate it! |
|
Gentle ping — this is a small docs-only fix and all checks are green. Would a core dev be able to take a look when you have a moment? Happy to make any changes. Thanks! |
|
Sorry we missed this, it was fixed in 49aed47. |
The default compresslevel for gzip.open(), the GzipFile constructor, and gzip.compress() is COMPRESS_LEVEL_TRADEOFF (6). The signatures and the "Changed in version" notes in Doc/library/gzip.rst already reflect this (it says "the default compression level was reduced to 6, down from 9").
However, a few places still state the default is 9 and should be updated:-