IntersectMBO / IntersectMBO/ouroboros-consensus

Remove dead NTN and NTC versions in Consensus

Open
#572 1 comment 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

As of https://github.com/input-output-hk/ouroboros-network/pull/3699 and https://github.com/input-output-hk/ouroboros-network/pull/3696, there will be dead code in Consensus, such as `CardanoNodeToNodeVersionX` for X<5 etc.

This ticket is to delete those now-dead definitions and whatever that transitively kills, and so on.

# Acceptance criteria
We've removed all the dead-code and updated the `interface-CHANGELOG.md` (if there's anything to add there).

Contributor guide

Open the contributing guide

Research direction

Start by locating the dead NTN and NTC version definitions in Consensus, including CardanoNodeToNodeVersionX for X<5, and trace the code they make unreachable. Remove the dead code and update interface-CHANGELOG.md if required; completion means all such dead code is gone.

Written by the indexing model from the issue text.

Assessment

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

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.