Skip to content

Bump the (minimum) protocol version to avoid invoking legacy behavior#1119

Merged
zkbot merged 1 commit intozcash:zc.v0.11.2.latestfrom
ebfull:version-bump
Jul 21, 2016
Merged

Bump the (minimum) protocol version to avoid invoking legacy behavior#1119
zkbot merged 1 commit intozcash:zc.v0.11.2.latestfrom
ebfull:version-bump

Conversation

@ebfull
Copy link
Copy Markdown
Contributor

@ebfull ebfull commented Jul 20, 2016

This sets the current protocol version to 170002 (an order of magnitude larger than Bitcoin's) and the minimum to the same.

Closes #1107

@ebfull ebfull added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Jul 20, 2016
@daira
Copy link
Copy Markdown
Contributor

daira commented Jul 20, 2016

ACK

@daira
Copy link
Copy Markdown
Contributor

daira commented Jul 20, 2016

Note that we should bump these again (by 1) for each release after z7 in which we make incompatible protocol changes.

@ebfull
Copy link
Copy Markdown
Contributor Author

ebfull commented Jul 21, 2016

@zkbot r+

@zkbot
Copy link
Copy Markdown
Contributor

zkbot commented Jul 21, 2016

📌 Commit 697cc53 has been approved by ebfull

@zkbot
Copy link
Copy Markdown
Contributor

zkbot commented Jul 21, 2016

⌛ Testing commit 697cc53 with merge cebeabb...

zkbot pushed a commit that referenced this pull request Jul 21, 2016
Bump the (minimum) protocol version to avoid invoking legacy behavior

This sets the current protocol version to `170002` (an order of magnitude larger than Bitcoin's) and the minimum to the same.

Closes #1107
@zkbot
Copy link
Copy Markdown
Contributor

zkbot commented Jul 21, 2016

☀️ Test successful - zcash

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

Labels

S-waiting-on-review Status: Awaiting review from the assignee but also interested parties.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants