Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Tests for the wrap_result functionality. #15

Merged
merged 1 commit into from Jan 24, 2014

Conversation

Projects
None yet
2 participants
Contributor

TheWatcher commented Jan 24, 2014

As requested, tests for the wrap_result feature, please tell me if you consider this adequate coverage.

I've been using the new feature in my project, and it is working out great so far!

semifor added a commit that referenced this pull request Jan 24, 2014

Merge pull request #15 from TheWatcher/wrap-result
Tests for the wrap_result functionality.

@semifor semifor merged commit 807aa4a into semifor:wrap-result Jan 24, 2014

Owner

semifor commented Jan 24, 2014

Thank you! I added a CONTRIBUTORS section to lib/Net/Twitter/Lite.pod. Care to add an attribution there before I release?

Contributor

TheWatcher commented Jan 24, 2014

I'm apparently blind, but I can't see a CONTRIBUTORS section in there.

But anyway, I'd be happy for you to release without any attribution; you did the hard work after all!

Owner

semifor commented Jan 24, 2014

Ugh! My apologies. I didn't push the commit with that change. :-/

It's ready for you, now.

On Fri, Jan 24, 2014 at 3:54 PM, Chris notifications@github.com wrote:

I'm apparently blind, but I can't see a CONTRIBUTORS section in there.

But anyway, I'd be happy for you to release without any attribution; you
did the hard work after all!


Reply to this email directly or view it on GitHubhttps://github.com/semifor/net-twitter-lite/pull/15#issuecomment-33273173
.

Contributor

TheWatcher commented Jan 25, 2014

Heh, no worries, easy done. Sent #16 with addition.

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