[ecashaddrjs] Support ecregtest prefix
ClosedPublic
Actions

Authored by bytesofman on Apr 11 2023, 14:03.

Details

Reviewers

Fabien

Group Reviewers

Restricted Project

Commits

rABCdbd84c022d0c: [ecashaddrjs] Support ecregtest prefix

Summary

T3106

Add ecregtest as a valid prefix. Do not add it to VALID_PREFIXES_MAINNET

Test Plan

I did not add any specific unit tests for this prefix (similar to bchreg prefix). However, npm test to confirm existing functionality is not impacted.

Diff Detail

Repository

rABC Bitcoin ABC

Branch

add-ecregtest

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 23199
Build 46017: Build Diff
Build 46016: arc lint + arc unit

Event Timeline

bytesofman created this revision.Apr 11 2023, 14:03

Herald added a reviewer: Restricted Project. · View Herald TranscriptApr 11 2023, 14:03

bytesofman requested review of this revision.Apr 11 2023, 14:03

Harbormaster completed remote builds in B23199: Diff 39543.Apr 11 2023, 14:03

Fabien accepted this revision.Apr 11 2023, 14:17

This revision is now accepted and ready to land.Apr 11 2023, 14:17

Closed by commit rABCdbd84c022d0c: [ecashaddrjs] Support ecregtest prefix (authored by bytesofman). · Explain WhyApr 11 2023, 15:48

This revision was automatically updated to reflect the committed changes.

bytesofman added a commit: rABCdbd84c022d0c: [ecashaddrjs] Support ecregtest prefix.

Revision Contents
Changeset List

Path

Size

modules/

ecashaddrjs/

src/

cashaddr.js

1 line

Diff 39543

View Options

modules/ecashaddrjs/src/cashaddr.js

[ecashaddrjs] Support ecregtest prefixClosedPublicActions