[ecash-coinselect] Remove double defined const
ClosedPublic
Actions

Authored by bytesofman on Feb 27 2024, 00:30.

Details

Reviewers

emack

Group Reviewers

Restricted Project

Commits

rABC5b3fc0f21b28: [ecash-coinselect] Remove double defined const

Summary

This is defined at the top of the file and does not need to be specifically defined in one test

no version bump here, just code cleanup

Test Plan

npm test

Diff Detail

Repository

rABC Bitcoin ABC

Branch

coinselect-oops

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 27454
Build 54471: Build Diff	ecash-coinselect
Build 54470: arc lint + arc unit

Event Timeline

bytesofman created this revision.Feb 27 2024, 00:30

Herald added a reviewer: Restricted Project. · View Herald TranscriptFeb 27 2024, 00:30

bytesofman published this revision for review.Feb 27 2024, 00:30

Harbormaster completed remote builds in B27454: Diff 45653.Feb 27 2024, 01:02

emack accepted this revision.Feb 27 2024, 07:04

This revision is now accepted and ready to land.Feb 27 2024, 07:04

Closed by commit rABC5b3fc0f21b28: [ecash-coinselect] Remove double defined const (authored by bytesofman). · Explain WhyFeb 27 2024, 11:58

This revision was automatically updated to reflect the committed changes.

bytesofman added a commit: rABC5b3fc0f21b28: [ecash-coinselect] Remove double defined const.

Revision Contents
Changeset List

Path

Size

modules/

ecash-coinselect/

test/

coinSelect.test.js

3 lines

Diff 45653

View Options

modules/ecash-coinselect/test/coinSelect.test.js

[ecash-coinselect] Remove double defined constClosedPublicActions