IntersectMBO / IntersectMBO/ouroboros-consensus

Remove dummy codec for the obsolete `SafeBeforeEpoch` data type

Open
#1,589 0 comments 0 reactions 0 assignees View on GitHub
technical debt
Dominant language
Haskell
Stars
67
Forks
43
Avg merge
5d 13h
Merged PRs (30d)
43

Description

This data type was removed in https://github.com/IntersectMBO/ouroboros-network/issues/2646.

From a comment by @amesgen in https://github.com/IntersectMBO/ouroboros-consensus/pull/1588#discussion_r2207150096:

> I guess a low-prio thing to do would be to actually fully remove this artifact, in a similar way as we did for when `GenesisWindow` was added to `EraParams` in 3fc6d28f00b0a1b0d3187cb7916f1e192dd79fcb (with the compat code removed in e477ed0e5dc5d9cbdac60e8824a7bda411d550b4).

Contributor guide

Open the contributing guide

Research direction

No file or test is named. Start by locating references to the obsolete SafeBeforeEpoch data type and its dummy codec, then compare the cleanup approach in commits 3fc6d28f00b0a1b0d3187cb7916f1e192dd79fcb and e477ed0e5dc5d9cbdac60e8824a7bda411d550b4. Done means the artifact and related compatibility code are fully removed without remaining references.

Written by the indexing model from the issue text.

Assessment

Tech stack
haskell
Domain
blockchain
Issue type
Refactor
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.