FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

gh-91279: Note ``SOURCE_DATE_EPOCH`` support in ``ZipFile.writestr()`` by Wulian233 · Pull Request #139396 · python/cpython · GitHub

/ cpython Public

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

Merged
vstinner merged 10 commits into
python:mainfrom
Wulian233:doc
Mar 20, 2026
ConversationCommits (10)ChecksFiles changed
Merged

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

Conversation

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

Copy link
Copy Markdown
Contributor

bedevere-app Bot added awaiting review docs Documentation in the Doc dir skip news labels Sep 28, 2025
github-project-automation Bot moved this to Todo in Docs PRs Sep 28, 2025
bedevere-app Bot mentioned this pull request Sep 28, 2025
Closed

This comment was marked as outdated.

Sign in to view

This comment was marked as outdated.

Sign in to view

This comment was marked as outdated.

Sign in to view

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

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

Wulian233 commented Oct 3, 2025

Copy link
Copy Markdown
Contributor Author

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

Finished, thank you!

AA-Turner previously requested changes Oct 4, 2025
View reviewed changes
Comment thread Doc/library/zipfile.rst Outdated
AA-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
Wulian233 requested a review from AA-Turner October 11, 2025 12:07
vstinner 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-app Bot mentioned this pull request Nov 4, 2025
Closed
vstinner reviewed Nov 4, 2025
View reviewed changes
Comment thread Doc/library/zipfile.rst Outdated
Update Doc/library/zipfile.rst
217b831 
Co-authored-by: Victor Stinner <vstinner@python.org>
vstinner approved these changes Nov 4, 2025
View reviewed changes

vstinner 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.

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

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

Wulian233 commented Mar 20, 2026

Copy link
Copy Markdown
Contributor Author

ping @vstinner @AA-Turner

vstinner added the needs backport to 3.14 bugs and security fixes label Mar 20, 2026
vstinner reviewed Mar 20, 2026
View reviewed changes
Comment thread Doc/library/zipfile.rst Outdated
vstinner dismissed AA-Turner’s stale review March 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 enabled auto-merge (squash) March 20, 2026 12:09

Wulian233 commented Mar 20, 2026

Copy link
Copy Markdown
Contributor Author

thanks!

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 merged commit 5ad738f into python:main Mar 20, 2026
28 checks passed
github-project-automation Bot moved this from Todo to Done in Docs PRs Mar 20, 2026

miss-islington-app Bot 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
pythongh-91279: Note SOURCE_DATE_EPOCH support 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 Bot commented Mar 20, 2026

Copy link
Copy Markdown

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

bedevere-app Bot removed the needs backport to 3.14 bugs and security fixes label Mar 20, 2026

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
[3.14] gh-91279: Note SOURCE_DATE_EPOCH support 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 deleted the doc branch March 20, 2026 22:02
CuriousLearner added a commit to CuriousLearner/cpython that referenced this pull request Mar 22, 2026
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)
  ...
pablogsal mentioned this pull request Apr 4, 2026
Merged
ljfp pushed a commit to ljfp/cpython that referenced this pull request Apr 25, 2026
pythongh-91279: Note SOURCE_DATE_EPOCH support in `ZipFile.writestr…
7cb649a 
…()` doc (python#139396)

Co-authored-by: Victor Stinner <vstinner@python.org>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Show hidden characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

vstinner vstinner approved these changes
AA-Turner Awaiting requested review from AA-Turner

Assignees

No one assigned

Labels

docs Documentation in the Doc dir skip news

Projects

Docs PRs
Status: Done

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

Back | FazBrowse Home | New Git URL