Skip to content

Conversation

@Crydust
Copy link
Contributor

@Crydust Crydust commented Feb 20, 2014

No description provided.

@Crydust
Copy link
Contributor Author

Crydust commented Mar 13, 2014

Why is this not merged in yet?
Is there something wrong with the pull request?
I'm willing to rework it if needed.

@stupenrose
Copy link
Contributor

Sorry, for the slow response - very busy at work. This looks good, but I think we're at a point with the mojo where I'm reticent to add new functionality unless we have corresponding test coverage; I'd like to preserve our ability to crank out new releases with minimal work (i.e. without having to go through a manual testing episode/phase). So, long story short: can you please submit a test?

@Crydust
Copy link
Contributor Author

Crydust commented Mar 27, 2014

Ok.
Also swamped, but will look into it.

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