Page MenuHomePhabricator
Differential D9805

Rebrand the comments in wallet_avoidreuse.py
ClosedPublic
Actions

Authored by Fabien on Jul 19 2021, 15:19.

Details

Reviewers
majcosta
Group Reviewers
Restricted Project
Maniphest Tasks
T1672: Node software rebrand
Commits
rABCa9872acb6f0d: Rebrand the comments in wallet_avoidreuse.py
Summary

This one has a lot of comments affected, so put it into review alone.

Ref T1672.

Test Plan
./test/functional/test_runner.py wallet_avoidreuse.py

Read the comments.

Diff Detail

Repository
rABC Bitcoin ABC
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

Fabien created this revision.Jul 19 2021, 15:19
Herald added a reviewer: Restricted Project. · View Herald TranscriptJul 19 2021, 15:20
Fabien requested review of this revision.Jul 19 2021, 15:20
Harbormaster completed remote builds in B16211: Diff 29236.Jul 19 2021, 15:38
majcosta requested changes to this revision.Jul 19 2021, 18:36
majcosta added a subscriber: majcosta.

what about abbreviating the million? I read that 'MM' is most common than 'm' or 'mil' so, use 1MM XEC instead of 1,000,000 seems to nicely avoid unnecessary zeroes

This revision now requires changes to proceed.Jul 19 2021, 18:36
Fabien updated this revision to Diff 29246.Jul 20 2021, 07:27

Use MM for millions

Harbormaster completed remote builds in B16217: Diff 29246.Jul 20 2021, 07:41
majcosta accepted this revision.Jul 20 2021, 14:45
This revision is now accepted and ready to land.Jul 20 2021, 14:45
Closed by commit rABCa9872acb6f0d: Rebrand the comments in wallet_avoidreuse.py (authored by Fabien). · Explain WhyJul 20 2021, 14:56
This revision was automatically updated to reflect the committed changes.
Fabien added a commit: rABCa9872acb6f0d: Rebrand the comments in wallet_avoidreuse.py.

Revision Contents
Changeset List

PathSize
test/
functional/
83 lines

Diff 29269

View Options

test/functional/wallet_avoidreuse.py

Loading...