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

Remove 20% founders reward #1714

Closed
wants to merge 2 commits into
base: master
from

Conversation

Projects
None yet
4 participants
@heyrhett
Contributor

heyrhett commented Oct 29, 2016

Hi Team,

Congratulations on the great work and the mining rewards that you earned today!

This patch removes further founders rewards and instead lets miners keep all of their rewards for running the network.

@daira

This comment has been minimized.

Contributor

daira commented Oct 29, 2016

A Founders' Reward removal fork would probably want to remove dead code such as GetLastFoundersRewardBlockHeight, the lists of addresses in chainparams.cpp, etc. However, forks should be developed in their own repos with separate bug trackers.

@@ -1,6 +1,10 @@
Zcash 1.0.0
Zcash Classic 1.0.0
===========

This comment has been minimized.

@daira

daira Oct 29, 2016

Contributor

Markdown requires heading underlines to be at least as long as the heading :-)

This comment has been minimized.

@heyrhett

heyrhett Oct 29, 2016

Contributor

Great, now you are sending me pull requests :-) Merged!

This comment has been minimized.

@daira

daira Oct 29, 2016

Contributor

Note, however, that "Zcash" is a trademark, so you'll need to call it something else.

@Frostcoin

This comment has been minimized.

Frostcoin commented Oct 30, 2016

acc

@str4d

This comment has been minimized.

Contributor

str4d commented Nov 2, 2016

I have deleted several comments on this thread that breached the Contributor Code of Conduct.

@zcash zcash locked and limited conversation to collaborators Nov 2, 2016

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