Skip to content

BUG: fix the method for checking local files for 1.24.x#23739

Merged
charris merged 2 commits intonumpy:maintenance/1.24.xfrom
LoveEatCandy:1.24.x/check_local_file
May 14, 2023
Merged

BUG: fix the method for checking local files for 1.24.x#23739
charris merged 2 commits intonumpy:maintenance/1.24.xfrom
LoveEatCandy:1.24.x/check_local_file

Conversation

@LoveEatCandy
Copy link
Copy Markdown

@LoveEatCandy LoveEatCandy commented May 9, 2023

Backport of #23728.

BufferedReader and BufferedWriter cannot be used to determine local files. For example, users can implement CustomFile to operate on OSS files, and then use BufferedReader(CustomFile) to achieve the buffered effect. But fileno method can do it.

@seberg This PR is for 1.24.x backport, thank you for support python3.8

@seberg seberg added the 08 - Backport Used to tag backport PRs label May 9, 2023
@seberg
Copy link
Copy Markdown
Member

seberg commented May 9, 2023

Oops, test failure.

@LoveEatCandy
Copy link
Copy Markdown
Author

Strange, I pushed the code to the branch but the PR didn't update.What should i do?

@seberg
Copy link
Copy Markdown
Member

seberg commented May 9, 2023

Just a github hickup, I am sure: https://www.githubstatus.com try again later.

@mattip
Copy link
Copy Markdown
Member

mattip commented May 9, 2023

Can you push an empty commit to retrigger github?

@LoveEatCandy
Copy link
Copy Markdown
Author

Can you push an empty commit to retrigger github?

Done

@mattip
Copy link
Copy Markdown
Member

mattip commented May 9, 2023

CI passes except for s390x which did not even start (not the fault of this PR).

@charris charris merged commit 512bd17 into numpy:maintenance/1.24.x May 14, 2023
@charris
Copy link
Copy Markdown
Member

charris commented May 14, 2023

Thanks @LoveEatCandy .

@LoveEatCandy LoveEatCandy deleted the 1.24.x/check_local_file branch May 15, 2023 06:24
@charris charris added this to the 1.24.4 release milestone Jun 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

00 - Bug 08 - Backport Used to tag backport PRs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants