Skip to content
This repository has been archived by the owner on Feb 10, 2021. It is now read-only.

Add isdir and isfile #140

Merged
merged 2 commits into from
Nov 13, 2017
Merged

Add isdir and isfile #140

merged 2 commits into from
Nov 13, 2017

Conversation

jcrist
Copy link
Member

@jcrist jcrist commented Nov 11, 2017

Used to better match the interface required by arrow.

Used to better match the interface required by arrow.
@martindurant
Copy link
Member

Apparently you should account for OSError and IOError when writing for both py2 and py3.

@pitrou
Copy link
Member

pitrou commented Nov 13, 2017

Apparently you should account for OSError and IOError when writing for both py2 and py3.

Just catch EnvironmentError, which is a base class (or an alias) for both.

@martindurant martindurant merged commit 5fda481 into dask:master Nov 13, 2017
@jcrist jcrist deleted the isdir-isfile branch November 13, 2017 16:56
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants