/bitcoin.org

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

New softfork description #1436

Merged
merged 2 commits into from Feb 13, 2017

Conversation

Reviewers
No reviews
Assignees

@wbnns wbnns

Labels
Merge Scheduled
Projects
None yet
Milestone
No milestone
3 participants
@Mirobit @luke-jr @wbnns
@Mirobit
Contributor

Mirobit commented Dec 14, 2016

As suggested in #1431
Not happy with the description but better than the old one and options are quite limited with 255 chars.

@Mirobit Mirobit referenced this pull request Dec 14, 2016

Closed

Soft Fork Definition #1431

@luke-jr
Contributor

luke-jr commented Dec 14, 2016

ACK after removing the last sentence. If only miners enforce a softfork, something is wrong.

@wbnns wbnns self-assigned this Dec 16, 2016

@wbnns wbnns added the Under Review label Dec 16, 2016

@wbnns wbnns added On Hold Changes Requested and removed Under Review labels Feb 9, 2017

@Mirobit
Contributor

Mirobit commented Feb 9, 2017

@luke-jr Agreed. While technical true, it is not desirable.

@wbnns wbnns added Merge Scheduled and removed Changes Requested On Hold labels Feb 9, 2017

@wbnns
Contributor

wbnns commented Feb 9, 2017

@Mirobit @luke-jr Thanks guys.

Unless others object, this will be merged on Sunday, February 12th.

@wbnns wbnns merged commit 20a8f34 into bitcoin-dot-org:master Feb 13, 2017
1 check passed

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details

@Mirobit Mirobit deleted the Mirobit:softforkdef branch May 5, 2017

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