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

GH-127381: pathlib ABCs: remove `PathBase.stat()` by barneygale · Pull Request #128334 · python/cpython · GitHub

/ cpython Public

GH-127381: pathlib ABCs: remove PathBase.stat() - #128334

Merged
barneygale merged 2 commits into
python:mainfrom
barneygale:gh-127381-stat
Dec 29, 2024
Merged

GH-127381: pathlib ABCs: remove PathBase.stat()#128334
barneygale merged 2 commits into
python:mainfrom
barneygale:gh-127381-stat

Conversation

barneygale commented Dec 29, 2024
edited
Loading

Copy link
Copy Markdown
Contributor

Remove the PathBase.stat() method. Its use of the os.stat_result API, with its 10 mandatory fields and low-level types, makes it an awkward fit for virtual filesystems.

We'll look to add a PathBase.info attribute later - see GH-125413.

No user-facing changes because the ABCs are still private.

Remove the `PathBase.stat()` method. Its use of the `os.stat_result` API,
with its 10 mandatory fields and low-level types, makes it a poor fit for
virtual filesystems.

We'll look to add a `PathBase.info` attribute later - see pythonGH-125413.
barneygale enabled auto-merge (squash) December 29, 2024 20:51
barneygale merged commit c78729f into python:main Dec 29, 2024
srinivasreddy pushed a commit to srinivasreddy/cpython that referenced this pull request Jan 8, 2025
Remove the `PathBase.stat()` method. Its use of the `os.stat_result` API,
with its 10 mandatory fields and low-level types, makes it an awkward fit
for virtual filesystems.

We'll look to add a `PathBase.info` attribute later - see pythonGH-125413.
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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant


Back | FazBrowse Home | New Git URL