Skip to content

chore: Remove unused code across multiple modules#10443

Closed
Copilot wants to merge 1 commit intorefactoring/unused-codefrom
copilot/sub-pr-10440
Closed

chore: Remove unused code across multiple modules#10443
Copilot wants to merge 1 commit intorefactoring/unused-codefrom
copilot/sub-pr-10440

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Feb 7, 2026

Changes

Removed 287 lines of dead code across Core, Network, State, Trie, Consensus, Serialization, and chain-specific modules:

  • Deleted unused helper classes: HashHelpers, InsertionBehavior, PeerEqualityComparer
  • Removed orphaned XDC infrastructure: IXdcSealer, QuorumCertificateException, XdcDbNames
  • Deleted unused SSZ container serialization utilities
  • Cleaned up unused extension methods in BlockHeaderExtensions and PatriciaTree
  • Removed dead fields and using directives across affected modules

Types of changes

What types of changes does your code introduce?

  • Bugfix (a non-breaking change that fixes an issue)
  • New feature (a non-breaking change that adds functionality)
  • Breaking change (a change that causes existing functionality not to work as expected)
  • Optimization
  • Refactoring
  • Documentation update
  • Build-related changes
  • Other: Description

Testing

Requires testing

  • Yes
  • No

Documentation

Requires documentation update

  • Yes
  • No

Requires explanation in Release Notes

  • Yes
  • No

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Copilot AI changed the title [WIP] Refactor unused code for optimization chore: Remove unused code across multiple modules Feb 7, 2026
Copilot AI requested a review from benaadams February 7, 2026 18:03
@benaadams benaadams closed this Feb 7, 2026
@benaadams benaadams deleted the copilot/sub-pr-10440 branch February 7, 2026 18:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants