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

fix #3889 #3891

Merged
merged 1 commit into from Aug 4, 2013
Merged

fix #3889 #3891

merged 1 commit into from Aug 4, 2013

Conversation

ivanov
Copy link
Member

@ivanov ivanov commented Aug 4, 2013

No description provided.

@ivanov
Copy link
Member Author

ivanov commented Aug 4, 2013

this should fix #3889

@ivanov
Copy link
Member Author

ivanov commented Aug 4, 2013

reporter of #3889 confirmed this fixes the test failure, ready to merge.

@Carreau
Copy link
Member

Carreau commented Aug 4, 2013

+1

@ivanov
Copy link
Member Author

ivanov commented Aug 4, 2013

push the button, Matthias, I dare you to try and break master again ;)

@Carreau
Copy link
Member

Carreau commented Aug 4, 2013

Would love, no merge button on phone though.

Please break it for me. But put your spandex on first.

ivanov added a commit that referenced this pull request Aug 4, 2013
@ivanov ivanov merged commit c2cabd0 into ipython:master Aug 4, 2013
@ivanov ivanov deleted the fix-3889 branch August 4, 2013 14:22
mattvonrocketstein pushed a commit to mattvonrocketstein/ipython that referenced this pull request Nov 3, 2014
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.

None yet

2 participants