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

Add rake and bundler tasks #1

Merged
merged 1 commit into from
Jun 30, 2017

Conversation

techpeace
Copy link
Contributor

@techpeace techpeace commented Jun 28, 2017

I noticed this library hadn't been pushed to RubyGems quite yet, so I took the liberty since I needed it for a project I'm working on.

This PR adds rake as a dependency, along with the necessary tasks to publish the gem. Let me know if you'd like me to transfer ownership of the gem over to you.

Thanks for the great lib!

This enables publishing to RubyGems.
@scarhand scarhand merged commit 32073c9 into phusion:master Jun 30, 2017
@scarhand
Copy link
Member

Thanks for the PR and for making the effort to publish it on RubyGems! It would be awesome if you could transfer ownership of the gem to me.

@FooBarWidget
Copy link
Member

Hi @techpeace and thanks for the contribution. We wrote this gem mostly for internal purposes and didn't think others would find it useful, so it's good that you've found this gem valuable. Having said this, we neglected to put a license on this gem. Would you agree to apply the MIT license to your code contributions?

@techpeace
Copy link
Contributor Author

Oh sorry about that @FooBarWidget, I neglected to check the license! Yes indeed, I agree to apply the MIT license to my contributions.

Sounds good @scarhand, I'll transfer it your way right now.

@techpeace
Copy link
Contributor Author

@scarhand: Just realized I'll need the email address you'd like me to use for the new gem owner on RubyGems. Thanks!

@scarhand
Copy link
Member

scarhand commented Jul 6, 2017

@techpeace oh yes, of course. Please use niels@pharynx.nl, thanks! As you might have noticed, I've also added the license.

@techpeace
Copy link
Contributor Author

Apologies for the delay, @scarhand - transferred your way!

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.

3 participants