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 test for ticket #16731. #1117

Closed
wants to merge 2 commits into
base: master
from

Conversation

Projects
None yet
2 participants
@kvsn

kvsn commented May 18, 2013

No description provided.

@mjtamlyn

This comment has been minimized.

Show comment
Hide comment
@mjtamlyn

mjtamlyn May 19, 2013

Member

Closing for the reasons expressed on the ticket. Thanks for the contribution though, it's still very helpful!

Thanks for your contribution. We can't merge this in without the fix, it's not quite the purpose of expectedFailure. I'll close the PR for now, I've added your failing test as a patch file to this ticket for posterity.

Member

mjtamlyn commented May 19, 2013

Closing for the reasons expressed on the ticket. Thanks for the contribution though, it's still very helpful!

Thanks for your contribution. We can't merge this in without the fix, it's not quite the purpose of expectedFailure. I'll close the PR for now, I've added your failing test as a patch file to this ticket for posterity.

@mjtamlyn mjtamlyn closed this May 19, 2013

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment