Skip to content

Conversation

@onbjerg
Copy link
Contributor

@onbjerg onbjerg commented Oct 22, 2025

Motivation

These functions seem completely unused, even at their introduction in #10226

Solution

Remove them.

PR Checklist

  • Added Tests
  • Added Documentation
  • Breaking changes

Copy link
Member

@zerosnacks zerosnacks left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks like a leftover, resolving is now done w/ alloy-chains

Copy link
Contributor

@0xrusowsky 0xrusowsky left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm if CI is happy

@onbjerg onbjerg enabled auto-merge October 22, 2025 12:48
@onbjerg onbjerg added this pull request to the merge queue Oct 22, 2025
Merged via the queue into master with commit 73f0977 Oct 22, 2025
27 of 29 checks passed
@onbjerg onbjerg deleted the push-uownltqquztv branch October 22, 2025 13:27
@github-project-automation github-project-automation bot moved this to Done in Foundry Oct 22, 2025
@grandizzy grandizzy moved this from Done to Completed in Foundry Oct 27, 2025
@rplusq rplusq mentioned this pull request Oct 31, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

T-debt Type: code debt

Projects

Status: Completed

Development

Successfully merging this pull request may close these issues.

5 participants