Page MenuHomePhabricator

Update README and Release notes for 0.14.3
ClosedPublic

Authored by Mengerian on Jul 11 2017, 22:33.

Details

Reviewers
deadalnix
freetrader
sickpig
Group Reviewers
Restricted Owners Package(Owns No Changed Paths)
Restricted Project
Commits
rSTAGING2ba4f7c0edb4: Update README and Release notes for 0.14.3
rABC2ba4f7c0edb4: Update README and Release notes for 0.14.3
Summary

Change version numbers for ABC 0.14.3 release:

configure.ac
doc/README.md
doc/release-notes.md
src/test/net_tests.cpp

Move older release notes to doc/release-notes/

Test Plan

None

Diff Detail

Repository
rABC Bitcoin ABC
Branch
rel-14-2
Lint
Lint Passed
SeverityLocationCodeMessage
Auto-Fixsrc\test\net_tests.cpp:1CFMTCode style violation
Unit
No Test Coverage
Build Status
Buildable 512
Build 512: arc lint + arc unit

Event Timeline

Owners added a reviewer: Restricted Owners Package.Jul 11 2017, 22:33
doc/release-notes.md
5 ↗(On Diff #794)

This is including several mistakes from the original content.

Here's a rough attempt at correction:

This release include various changes. The most notable is a change to the difficulty adjustment algorithm which decreases the difficulty in case the hashrate becomes very low. We obviously hope that this feature will not need to be used, and it would not kick in if the hashrate stays above 8% of the global hashrate. However, it ensures that this chain will survive no matter what, as long as someone thinks it is valuable.

doc/release-notes.md
5 ↗(On Diff #794)

I missed another one in the first sentence:

This release includes various changes.

freetrader added inline comments.
doc/README.md
1 ↗(On Diff #794)

Would it be an opportune time to replace Core by ABC in this document?

6 ↗(On Diff #794)

One could rephrase to "Bitcoin ABC is a derivative of Bitcoin Core, the original ..."

This revision now requires changes to proceed.Jul 12 2017, 06:57
Mengerian edited edge metadata.

Updates based on @freetrader's comments.

Ended up fully De-Coreifyin the README.

Fixed wording of release notes based on freetrader's corrections

OK, I updated the diff to fix the issues identified by @freetrader

Fix download link in release notes

doc/README.md
1 ↗(On Diff #805)

0.14.3 now :) We missed the boat for 0.14.2 .

This revision is now accepted and ready to land.Jul 13 2017, 23:59
deadalnix requested changes to this revision.Jul 14 2017, 01:17

No please bump to 0.14.3 . Then it's good to go.

This revision now requires changes to proceed.Jul 14 2017, 01:17

No please bump to 0.14.3 . Then it's good to go.

In that case, may as well also change to 0.14.3 in configure.ca and /src/test/net_tests.cpp in this diff

Mengerian retitled this revision from Update README and Release notes for 0.14.2 to Update README and Release notes for 0.14.3.
Mengerian edited edge metadata.

Update for 0.14.3 release

sickpig added a subscriber: sickpig.

proof read, everything seems ok.

This revision is now accepted and ready to land.Jul 14 2017, 15:37
This revision was automatically updated to reflect the committed changes.