Skip to content

Implement new hardfork for v17 to be used in future changes - #3808

Merged
PastaPastaPasta merged 2 commits into
dashpay:developfrom
PastaPastaPasta:v17-hardfork
Nov 17, 2020
Merged

Implement new hardfork for v17 to be used in future changes#3808
PastaPastaPasta merged 2 commits into
dashpay:developfrom
PastaPastaPasta:v17-hardfork

Conversation

@PastaPastaPasta

Copy link
Copy Markdown
Member

Not sure I'm a huge fan of DEPLOYMENT_V17 name, lmk if there are any suggestions

Signed-off-by: pasta <pasta@dashboost.org>
@thephez

thephez commented Nov 11, 2020

Copy link
Copy Markdown
Collaborator

Will the fork primarily be to support platform requirements? If so, the name could reflect that.

@PastaPastaPasta PastaPastaPasta added this to the 17 milestone Nov 11, 2020
Comment thread src/consensus/params.h Outdated
@UdjinM6

UdjinM6 commented Nov 17, 2020

Copy link
Copy Markdown

Agree that a better name should be picked at some point (it's not even a HF atm) but we can tweak this later, it's not a blocker imo.

Co-authored-by: UdjinM6 <UdjinM6@users.noreply.github.com>

@UdjinM6 UdjinM6 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

utACK

@xdustinface xdustinface left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

utACK

@PastaPastaPasta
PastaPastaPasta merged commit a4d2543 into dashpay:develop Nov 17, 2020
@PastaPastaPasta
PastaPastaPasta deleted the v17-hardfork branch November 17, 2020 19:28
gades pushed a commit to cosanta/cosanta-core that referenced this pull request Mar 13, 2022
…3808)

* Implement new hardfork for v17 to be used in future changes

Signed-off-by: pasta <pasta@dashboost.org>

* fix comment

Co-authored-by: UdjinM6 <UdjinM6@users.noreply.github.com>

Co-authored-by: UdjinM6 <UdjinM6@users.noreply.github.com>
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.

4 participants