Page MenuHomePhabricator

[wallet] Remove CAccountingEntry class
ClosedPublic

Authored by nakihito on Mar 3 2020, 20:16.

Details

Reviewers
Fabien
Group Reviewers
Restricted Owners Package(Owns No Changed Paths)
Restricted Project
Commits
rSTAGINGaf67f6702662: [wallet] Remove CAccountingEntry class
rABCaf67f6702662: [wallet] Remove CAccountingEntry class
Summary
Test Plan
ninja check
test_runner.py wallet_*

Diff Detail

Repository
rABC Bitcoin ABC
Branch
PR13825-8
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 9714
Build 17321: Default Diff Build & Tests
Build 17320: arc lint + arc unit

Event Timeline

Owners added a reviewer: Restricted Owners Package.Mar 3 2020, 20:16

[Bot Message]
One or more PR numbers were detected in the summary.
Links to those PRs have been inserted into the summary for reference.

Fabien requested changes to this revision.Mar 4 2020, 08:17
Fabien added a subscriber: Fabien.

Please run the wallet functional tests as part of the test plan: ./test/functional/test_runner.py wallet_*. (wallet_backup is part of the extended tests).

This revision now requires changes to proceed.Mar 4 2020, 08:17
nakihito requested review of this revision.Mar 4 2020, 18:55
nakihito edited the test plan for this revision. (Show Details)

Updated test plan.

This revision is now accepted and ready to land.Mar 5 2020, 07:37
This revision was automatically updated to reflect the committed changes.