Skip to content

gh-90887: posix._fcopyfile adding couple of more flags #31300

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
JelleZijlstra merged 4 commits into from
May 3, 2022
Merged

gh-90887: posix._fcopyfile adding couple of more flags #31300

JelleZijlstra merged 4 commits into from
May 3, 2022

Conversation

@devnexen
Copy link
Contributor

@devnexen devnexen commented Feb 12, 2022 •
edited by JelleZijlstra
Loading

@devnexen devnexen changed the title posix._fcopyfile adding couple of more flags bpo-46731: posix._fcopyfile adding couple of more flags Feb 12, 2022
@JelleZijlstra
Copy link
Member

JelleZijlstra commented May 2, 2022

This needs a NEWS entry. I feel like it might also need to be documented in https://docs.python.org/3.10/library/os.html, but the existing flags aren't documented there either.

@devnexen devnexen changed the title bpo-46731: posix._fcopyfile adding couple of more flags gh-92172: posix._fcopyfile adding couple of more flags May 2, 2022
@devnexen devnexen force-pushed the fcopyfile_flags branch from 8d423f7 to 31a90f9 Compare May 2, 2022 18:32
@JelleZijlstra JelleZijlstra changed the title gh-92172: posix._fcopyfile adding couple of more flags gh-90887: posix._fcopyfile adding couple of more flags May 2, 2022
@JelleZijlstra JelleZijlstra self-assigned this May 2, 2022
@JelleZijlstra JelleZijlstra merged commit 04dc4b0 into python:main May 3, 2022
@devnexen devnexen mannequin mentioned this pull request Apr 10, 2022
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@JelleZijlstra JelleZijlstra JelleZijlstra approved these changes

Assignees

@JelleZijlstra JelleZijlstra

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@devnexen @JelleZijlstra @the-knights-who-say-ni @bedevere-bot