Skip to content

Conversation

@aisk
Copy link
Contributor

@aisk aisk commented Oct 4, 2025

@aisk
Copy link
Contributor Author

aisk commented Oct 4, 2025

I noticed a PR #139237 to migrate the mmap module to use AC. We should wait for it to be merged before continue the work.

@aisk aisk marked this pull request as draft October 4, 2025 12:26
@aisk aisk marked this pull request as ready for review October 11, 2025 13:00
@aisk
Copy link
Contributor Author

aisk commented Oct 11, 2025

Since #139237 was merged, I rebased the branch and now it's ready for review.

Copy link
Member

@vstinner vstinner left a comment

Choose a reason for hiding this comment

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

LGTM. Please just update the doc (see my comment).

Co-authored-by: Victor Stinner <vstinner@python.org>
@vstinner
Copy link
Member

Oh, you have to run make clinic to update Argument Clinic.

@aisk
Copy link
Contributor Author

aisk commented Dec 27, 2025

@vstinner Sorry I forgot check the CI status, updated!

@vstinner vstinner enabled auto-merge (squash) December 27, 2025 10:10
@vstinner vstinner merged commit 1af21ea into python:main Dec 27, 2025
46 checks passed
@aisk aisk deleted the mmap-flush-flags branch December 27, 2025 10:45
@picnixz
Copy link
Member

picnixz commented Dec 27, 2025

This broke some build bots: https://buildbot.python.org/#/builders/1232/builds/7610

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants