Skip to content
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.

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

add unlink to path API, fixes #99 #100

Merged
merged 2 commits into from
Nov 15, 2013
Merged

Conversation

AntoineD
Copy link
Contributor

No description provided.

tomerfiliba added a commit that referenced this pull request Nov 15, 2013
add unlink to path API, fixes #99
@tomerfiliba tomerfiliba merged commit 574f741 into tomerfiliba:master Nov 15, 2013
@tomerfiliba
Copy link
Owner

Oh, I haven't notice it... I only reviewed the first commit (7475550). I guess you probably pushed to the same branch as the original pull request and it slipped in

@AntoineD
Copy link
Contributor Author

I'm sorry about that :-(

@AntoineD
Copy link
Contributor Author

I assumed it was for python 2.5, I would have written it the modern way otherwise (see Issue #73 )

@tomerfiliba
Copy link
Owner

No problem. I forgot we still need this abomination for 2.5


Tomer Filiba
tomerfiliba.com http://www.facebook.com/tomerfiliba
http://il.linkedin.com/in/tomerfiliba

On Fri, Nov 15, 2013 at 11:39 PM, Antoine Dechaume <[email protected]

wrote:

I assumed it was for python 2.5, I would have written it the modern way
otherwise (see Issue #73#73)


Reply to this email directly or view it on GitHubhttps://github.com//pull/100#issuecomment-28605594
.

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.

2 participants