Skip to content

gh-91279: Note SOURCE_DATE_EPOCH support in ZipFile.writestr() - #139396

Merged
vstinner merged 10 commits into
python:mainfrom
Wulian233:doc
Mar 20, 2026
Merged

gh-91279: Note SOURCE_DATE_EPOCH support in ZipFile.writestr()#139396
vstinner merged 10 commits into
python:mainfrom
Wulian233:doc

Conversation

@Wulian233

@Wulian233Wulian233 commented Sep 28, 2025•
edited by bedevere-app Bot
Loading

Copy link
Copy Markdown
Contributor

@bedevere-appbedevere-appBot added awaiting review docs Documentation in the Doc dir skip news labels Sep 28, 2025
@bedevere-appbedevere-appBot mentioned this pull request Sep 28, 2025
Closed
AA-Turner

This comment was marked as outdated.

Sign in to view

@bedevere-app

This comment was marked as outdated.

Sign in to view
@Wulian233

This comment was marked as outdated.

Sign in to view
@AA-Turner

AA-Turner commented Oct 3, 2025

Copy link
Copy Markdown
Member

Ok, I'm proposing to remove that note from the 'A new type of interpreter' section, and I've also made the envvar change in the 'New & improved modules' PR. Could you retarget this PR to just make the zipfile.rst change?

A

@Wulian233Wulian233 changed the title gh-123299: Copyedit What's New in 3.14gh-123299: Copyedit What's New in zipfile.rstOct 3, 2025
@AA-TurnerAA-Turner changed the title gh-123299: Copyedit What's New in zipfile.rstgh-123299: Note SOURCE_DATE_EPOCH support in ZipFile.writestr()Oct 3, 2025
@Wulian233

Wulian233 commented Oct 3, 2025

Copy link
Copy Markdown
ContributorAuthor

Could you retarget this PR to just make the zipfile.rst change?

Finished, thank you!

AA-Turner
AA-Turner previously requested changes Oct 4, 2025
View reviewed changes
Comment threadDoc/library/zipfile.rst Outdated
@AA-TurnerAA-Turner changed the title gh-123299: Note SOURCE_DATE_EPOCH support in ZipFile.writestr()Note SOURCE_DATE_EPOCH support in ZipFile.writestr()Oct 4, 2025
@vstinnervstinner changed the title Note SOURCE_DATE_EPOCH support in ZipFile.writestr()gh-91279: Note SOURCE_DATE_EPOCH support in ZipFile.writestr()Nov 4, 2025
@bedevere-appbedevere-appBot mentioned this pull request Nov 4, 2025
Closed
vstinner
vstinner reviewed Nov 4, 2025
View reviewed changes
Comment threadDoc/library/zipfile.rst Outdated
@Wulian233@vstinner
Update Doc/library/zipfile.rst
217b831
Co-authored-by: Victor Stinner <vstinner@python.org>
vstinner
vstinner approved these changes Nov 4, 2025
View reviewed changes

@vstinnervstinner left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, but English is not my first language, so I'm not sure about the sentence :-)

@Wulian233

Wulian233 commented Mar 20, 2026

Copy link
Copy Markdown
ContributorAuthor

ping @vstinner@AA-Turner

@vstinnervstinner added the needs backport to 3.14 bugs and security fixes label Mar 20, 2026
vstinner
vstinner reviewed Mar 20, 2026
View reviewed changes
Comment threadDoc/library/zipfile.rst Outdated
@vstinner
vstinner dismissed AA-Turner’s stale reviewMarch 20, 2026 12:09

I think the text can be improved here. The sentence lacks an object, but more importantly we don't say what has actually changed in the method's behaviour if SDE is set.

The sentence was completed.

@vstinner
vstinner enabled auto-merge (squash) March 20, 2026 12:09
@Wulian233

Wulian233 commented Mar 20, 2026

Copy link
Copy Markdown
ContributorAuthor

thanks!

@vstinner

vstinner commented Mar 20, 2026

Copy link
Copy Markdown
Member

"All required checks pass" CI failed with a cryptic error message:

Error: The template is not valid. .github/workflows/build.yml (Line: 671, Col: 24): Error reading JToken from JsonReader. Path '', line 0, position 0.

So I updated the branch to attempt to fix the CI issue.

@vstinner
vstinner merged commit 5ad738f into python:mainMar 20, 2026
28 checks passed
@github-project-automationgithub-project-automationBot moved this from Todo to Done in Docs PRsMar 20, 2026
@miss-islington-app

miss-islington-appBot commented Mar 20, 2026

Copy link
Copy Markdown

Thanks @Wulian233 for the PR, and @vstinner for merging it 🌮🎉.. I'm working now to backport this PR to: 3.14.
🐍🍒⛏🤖 I'm not a witch! I'm not a witch!

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Mar 20, 2026
@Wulian233@vstinner@miss-islington
pythongh-91279: NoteSOURCE_DATE_EPOCHsupport in `ZipFile.writestr…
9cec510
…()` doc (pythonGH-139396)
(cherry picked from commit 5ad738f)
Co-authored-by: Wulian233 <1055917385@qq.com>
Co-authored-by: Victor Stinner <vstinner@python.org>
@bedevere-app

bedevere-appBot commented Mar 20, 2026

Copy link
Copy Markdown

GH-146222 is a backport of this pull request to the 3.14 branch.

@bedevere-appbedevere-appBot removed the needs backport to 3.14 bugs and security fixes label Mar 20, 2026
@vstinner

vstinner commented Mar 20, 2026

Copy link
Copy Markdown
Member

Merged, thanks for fixing the doc.

vstinner added a commit that referenced this pull request Mar 20, 2026
@miss-islington@Wulian233@vstinner
[3.14]gh-91279: NoteSOURCE_DATE_EPOCHsupport in `ZipFile.writest…
8b447fb
…r()` doc (GH-139396) (#146222)
gh-91279: Note `SOURCE_DATE_EPOCH` support in `ZipFile.writestr()` doc (GH-139396)
(cherry picked from commit 5ad738f)
Co-authored-by: Wulian233 <1055917385@qq.com>
Co-authored-by: Victor Stinner <vstinner@python.org>
@Wulian233
Wulian233 deleted the doc branch March 20, 2026 22:02
CuriousLearner added a commit to CuriousLearner/cpython that referenced this pull request Mar 22, 2026
@CuriousLearner
Merge remote-tracking branch 'upstream/main' into fix-pythongh-138577
2691ad9
* upstream/main: (1475 commits)
Docs: replace all `datetime` imports with `import datetime as dt` (python#145640)
pythongh-146153: Use `frozendict` in pure python fallback for `curses.has_key` (python#146154)
pythongh-138234: clarify returncode behavior for subprocesses created with `shell=True` (python#138536)
pythongh-140947: fix contextvars handling for server tasks in asyncio (python#141158)
pythonGH-100108: Add async generators best practices section (python#141885)
pythonGH-145667: Merge `GET_ITER` and `GET_YIELD_FROM_ITER` (pythonGH-146120)
pythongh-146228: Better fork support in cached FastPath (python#146231)
pythongh-146227: Fix wrong type in _Py_atomic_load_uint16 in pyatomic_std.h (pythongh-146229)
pythongh-145980: Fix copy/paste mistake in binascii.c (python#146230)
pythongh-146092: Raise MemoryError on allocation failure in _zoneinfo (python#146165)
pythongh-91279: Note `SOURCE_DATE_EPOCH` support in `ZipFile.writestr()` doc (python#139396)
pythongh-146196: Fix Undefined Behavior in _PyUnicodeWriter_WriteASCIIString() (python#146201)
pythongh-143930: Reject leading dashes in webbrowser URLs
pythongh-145916: Soft-deprecate ctypes.util.find_library (pythonGH-145919)
pythongh-146205: Check the errno with != 0 in close impls in select module (python#146206)
pythongh-146171: Fix nested AttributeError suggestions (python#146188)
pythongh-146099: Optimize _GUARD_CODE_VERSION+IP via function version symbols (pythonGH-146101)
pythongh-145980: Add support for alternative alphabets in the binascii module (pythonGH-145981)
pythongh-145754: Update signature retrieval in unittest.mock to use forwardref annotation format (python#145756)
pythongh-145177: Add emscripten run --test, uses test args from config.toml (python#146160)
...
@pablogsalpablogsal mentioned this pull request Apr 4, 2026
Merged
ljfp pushed a commit to ljfp/cpython that referenced this pull request Apr 25, 2026
@Wulian233@vstinner@ljfp
pythongh-91279: NoteSOURCE_DATE_EPOCHsupport in `ZipFile.writestr…
7cb649a
…()` doc (python#139396)
Co-authored-by: Victor Stinner <vstinner@python.org>
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@vstinnervstinnervstinner approved these changes
@AA-TurnerAA-TurnerAwaiting requested review from AA-Turner

Assignees

No one assigned

Labels

docsDocumentation in the Doc dirskip news

Projects

Docs PRs
Status: Done

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@Wulian233@AA-Turner@vstinner@nazeerali4325-commits