[Chronik] Remove `Network` proto enum
ClosedPublic
Actions

Authored by tobias_ruck on Apr 4 2023, 14:06.

Details

Reviewers

Fabien

Group Reviewers

Restricted Project

Commits

rABC9e7ee00a415d: [Chronik] Remove `Network` proto enum

Summary

We removed the network field in Tx in D13467 but not the enum.

Test Plan

ninja check-crates && ninja check-functional

Diff Detail

Repository

rABC Bitcoin ABC

Branch

chronik-remove-network-proto

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 22956
Build 45532: Build Diff	build-chronik
Build 45531: arc lint + arc unit

Event Timeline

tobias_ruck created this revision.Apr 4 2023, 14:06

Herald added a reviewer: Restricted Project. · View Herald TranscriptApr 4 2023, 14:06

tobias_ruck requested review of this revision.Apr 4 2023, 14:06

Good catch

This revision is now accepted and ready to land.Apr 4 2023, 14:23

Harbormaster completed remote builds in B22956: Diff 39227.Apr 4 2023, 14:48

Closed by commit rABC9e7ee00a415d: [Chronik] Remove `Network` proto enum (authored by tobias_ruck). · Explain WhyApr 4 2023, 15:59

This revision was automatically updated to reflect the committed changes.

tobias_ruck added a commit: rABC9e7ee00a415d: [Chronik] Remove `Network` proto enum.

Revision Contents
Changeset List

Path

Size

chronik/

chronik-proto/

proto/

chronik.proto

12 lines

Diff 39227

View Options

chronik/chronik-proto/proto/chronik.proto

[Chronik] Remove `Network` proto enumClosedPublicActions