Skip to content

[TECHDEBT] Consolidate & encapsulate ValidatorMap logic #203

@Olshansk

Description

@Olshansk

Objective

Encapsulate the concept of a ValidatorMap only to the consensus module

Origin Document

This discussion in #178:

Screen Shot 2022-09-11 at 3 15 53 PM

Goals

  • Remove all occurrences/references to the ValidatorMap throughout the codebase
  • Only maintain knowledge/understanding o the ValidatorMap in the consensus module

Deliverable

  • Do a grep for ValidatorMap throughout the whole codebase and it should only show up in the consensus module
  • A single PR refactoring / cleaning up this goal

Non-goals / Non-deliverables

  • Adding new functionality

General issue deliverables

  • Update the appropriate CHANGELOG
  • Update any relevant READMEs (local and/or global)
  • Update any relevant global documentation & references

Testing Methodology

  • All tests: make test_all
  • LocalNet: verify a LocalNet is still functioning correctly by following the instructions at docs/development/README.md

Creator: @Olshansk

Metadata

Metadata

Assignees

Labels

code healthNice to have code improvementconsensusConsensus specific changescoreCore infrastructure - protocol related

Type

No type

Projects

Status

Done

Relationships

None yet

Development

No branches or pull requests

Issue actions