-
-
Notifications
You must be signed in to change notification settings - Fork 18.6k
TST: win32 paths cannot be turned into URLs by prefixing them with "file://" v2 #5100
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
Conversation
@cpcloud for 0.13? |
@cpcloud 0.13? |
@mindw Can you rebase? We'd like to get this in for 0.13 (by next week). Thanks! |
rebase to where? |
Rebase to this PR. Assuming you're on the local branch associated with this PR: # add pandas upstream if you don't have it already
git remote add upstream git://github.com/pydata/pandas.git
# fetch upstream changes (do this anyway)
git fetch upstream
# rebase on top of upstream/master (this is current master)
git rebase upstream/master
# force push
git push --force You might need to resolve merge conflicts before force pushing. |
ping. |
@cpcloud pls review this, or can defer to 0.14....up2u |
bumping to 0.14, unless reviewed before |
I'll take it. |
I'd prefer the io/common name to be |
Also please rebase again and squash your commits together. |
Ok, please review. Why was it pushed to 0.14 again? |
return urljoin('file:', pathname2url(path)) | ||
|
||
|
||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
please remove this extra line.
Ok. |
this is fine, just remove the line. I can't merge this until you enable Travis though - looks like you had it set up months ago - see more here - https://github.com/pydata/pandas/blob/master/CONTRIBUTING.md#steps-to-enable-travis-ci |
I disabled it due to known test failures (test_fred?) |
those are fixed now, but can't merge this unless you set up Travis. |
Ok fixed. Waiting in line. -gabi On Tue, Oct 29, 2013 at 12:13 AM, Jeff Tratner notifications@github.comwrote:
|
TST: win32 paths cannot be turned into URLs by prefixing them with "file://" v2
thanks! |
Rebased on master (unable to use same pull request). Closes #4580.
see http://stackoverflow.com/questions/11687478/convert-a-filename-to-a-file-url