Page MenuHomePhabricator

[tests] Remove unused variables
ClosedPublic

Authored by nakihito on May 10 2019, 22:09.

Details

Reviewers
jasonbcox
deadalnix
Fabien
markblundeberg
Group Reviewers
Restricted Owners Package(Owns No Changed Paths)
Restricted Project
Commits
rSTAGINGfb6518323802: [tests] Remove unused variables
rABCfb6518323802: [tests] Remove unused variables
Summary

[tests] Remove unused variables

Backport of Core PR10047
https://github.com/bitcoin/bitcoin/pull/10047/
Commit 2/2

Depends on D3023

Test Plan
test_runner.py

Diff Detail

Repository
rABC Bitcoin ABC
Branch
PR10047-2
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 5812
Build 9686: Bitcoin ABC Buildbot (legacy)
Build 9685: arc lint + arc unit

Event Timeline

Owners added a reviewer: Restricted Owners Package.May 10 2019, 22:09
deadalnix requested changes to this revision.May 12 2019, 23:29

Where is part 1 ? Why doesn't this depends on part 1 ?

Also fix the test plan so command appear like command, prefixed with 2 spaces.

This revision now requires changes to proceed.May 12 2019, 23:29
nakihito edited the summary of this revision. (Show Details)EditedMay 13 2019, 00:10

Sorry, I must've forgotten to put the dependency in. Added now. Will fix the test plan later tonight.

nakihito edited the test plan for this revision. (Show Details)
This revision is now accepted and ready to land.May 16 2019, 15:50
This revision was automatically updated to reflect the committed changes.