Skip to content

Remove superseded commented-out method bodies in CustomSourcifyABILoader#158

Merged
marijamijailovic merged 1 commit into
mainfrom
rm/sourcify-loader-remove-old-impl
May 11, 2026
Merged

Remove superseded commented-out method bodies in CustomSourcifyABILoader#158
marijamijailovic merged 1 commit into
mainfrom
rm/sourcify-loader-remove-old-impl

Conversation

@mazurroman

@mazurroman mazurroman commented May 11, 2026

Copy link
Copy Markdown
Member

Summary

  • `CustomSourcifyABILoader` carried three large `/* ... */` blocks containing earlier implementations of `loadContract`, `getContract`, and `loadABI` that have been replaced by current implementations using Sourcify's `/v2/contract/...` endpoint. The old code is dead and never reached. Removes 112 lines of commented-out code.

Test plan

  • Active method implementations are unchanged
  • No other code references the removed legacy URLs/paths

@marijamijailovic marijamijailovic merged commit 254868e into main May 11, 2026
4 checks passed
@marijamijailovic marijamijailovic deleted the rm/sourcify-loader-remove-old-impl branch May 11, 2026 21:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@marijamijailovic marijamijailovic marijamijailovic approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@mazurroman @marijamijailovic