Fix for issue #560 #655

Merged
merged 2 commits into from Jul 2, 2012

Conversation

Projects
None yet
3 participants
Contributor

DavidMikeSimon commented Jun 19, 2012

No description provided.

Collaborator

andhapp commented Jun 19, 2012

@DavidMikeSimon: Please add specs related to this behaviour. It's not going to be merged in if there aren't any specs covering it. Thanks.

Will close the issue related to this.

Contributor

DavidMikeSimon commented Jun 19, 2012

Ok, will do.
On Jun 19, 2012 8:12 AM, "Anuj Dutta" <
reply@reply.github.com>
wrote:

@DavidMikeSimon: Please add specs related to this behaviour. It's not
going to be merged in if there aren't any specs covering it. Thanks.

Will close the issue related to this.


Reply to this email directly or view it on GitHub:
#655 (comment)

Contributor

DavidMikeSimon commented Jun 19, 2012

Okay, @andhapp, I've added a spec and verified that it only passes when my fix is in place.

Contributor

DavidMikeSimon commented Jun 28, 2012

Any further word on this pullreq? Is there anything else I need to do with it?

Collaborator

andhapp commented Jun 29, 2012

Will be merged next. Thanks.

ryanb added a commit that referenced this pull request Jul 2, 2012

@ryanb ryanb merged commit d20d90d into ryanb:master Jul 2, 2012

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