Skip to content

Conversation

@adamierymenko
Copy link

@adamierymenko adamierymenko commented Nov 8, 2017

Add credits for PR #6392

Description

This just adds a credit for us since we sponsored some development that's been merged in this cycle.

Add credits for PR rethinkdb#6392
@srh
Copy link
Contributor

srh commented Nov 12, 2017

It is correct that ZeroTier paid for this feature to be implemented, and not only that but financially it's one of the bigger community contributions, going by information in the recent Changelog interview.

If I remember correctly, DNSimple's providing free DNS services was explicitly conditional on them getting donor info put in the README. My guess is that DNSimple's marginal costs are a lot less than the combined previous profits they made when @mglukhovsky was paying for DNS.

I think it is appropriate that this be merged -- if there's a "Donors" section then donors should get put into it. Since I was the one getting paid I think it'd be gauche for me to merge this, because, you know, who am I to sell indulgences on behalf of the Church. You've got my stamp of approval on this PR, and if somebody else with merge power agrees, please merge.

@kofalt
Copy link

kofalt commented Nov 13, 2017

For what it's worth, as someone who's not getting paid, I think this should be merged as well 🙂
The linked PR looks great, and it's great to acknowledge parties who are contributing to the project!

@srh
Copy link
Contributor

srh commented Nov 17, 2017

Well, anybody who wanted to disagree had their chance! Merging.

@srh srh merged commit 137c207 into rethinkdb:next Nov 17, 2017
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