ZIP 1015: Specify development funding starting at NU6#881
Merged
Conversation
e03c4f2 to
357d145
Compare
…usly considerd" section. This also renumbers the remaining alternatives to reflect the moves, and updates the motivation for the 20% option such that it is does not depend heavily on context provided by the other alternatives.
…reviously considered alternatives. The Zcash community has selected Alternative 1, the Hybrid Deferred Dev Fund, for the behavior of the dev fund following the November 2024 halving. See the following posts for details: https://forum.zcashcommunity.com/t/dev-fund-decision-time/48231/16 https://forum.zcashcommunity.com/t/dev-fund-decision-time/48231/17 https://forum.zcashcommunity.com/t/dev-fund-decision-time/48231/27
9108f00 to
e20fe73
Compare
arya2
reviewed
Aug 26, 2024
arya2
requested changes
Aug 26, 2024
Collaborator
arya2
left a comment
There was a problem hiding this comment.
The changes to the protocol specification seem unintended, and we expected the draft ZIP to remain unchanged, everything else looks good. Reviewed with @conradoplg.
This includes some additional cleanup to definitions and wording in ZIP 1015 to clarify required references in ZIP 214.
e20fe73 to
e5dc216
Compare
daira
reviewed
Aug 26, 2024
daira
reviewed
Aug 26, 2024
daira
reviewed
Aug 26, 2024
conradoplg
approved these changes
Aug 26, 2024
Collaborator
conradoplg
left a comment
There was a problem hiding this comment.
LGTM pending current existing comments
arya2
approved these changes
Aug 26, 2024
Collaborator
arya2
left a comment
There was a problem hiding this comment.
LGTM pending existing comments
6e896a7 to
a3ee0f3
Compare
daira
reviewed
Aug 26, 2024
Co-authored-by: Daira-Emma Hopwood <daira@jacaranda.org>
a3ee0f3 to
57d0fe6
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This also renumbers the remaining alternatives to reflect the moves, and updates the motivation for the 20% option such that it does not depend heavily on context provided by the other alternatives.