Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

debt(foundry-common): Refactor ens utility to alloy #7127

Open
Evalir opened this issue Feb 14, 2024 · 2 comments
Open

debt(foundry-common): Refactor ens utility to alloy #7127

Evalir opened this issue Feb 14, 2024 · 2 comments
Labels
A-internals Area: internals D-chore Difficulty: chore T-debt Type: code debt
Milestone

Comments

@Evalir
Copy link
Member

Evalir commented Feb 14, 2024

#7122 introduces NameOrAddress which is capable of resolving an ens name to an address. Once providers on alloy are more stable (when alloy-rs/alloy#190 is merged), we should refactor this as an extension crate to alloy.

@Evalir Evalir added T-feature Type: feature D-chore Difficulty: chore labels Feb 14, 2024
@zerosnacks zerosnacks changed the title chore(foundry-common): Refactor ens utility to alloy debt(foundry-common): Refactor ens utility to alloy Jul 11, 2024
@zerosnacks zerosnacks added T-debt Type: code debt A-internals Area: internals and removed T-feature Type: feature labels Jul 11, 2024
@zerosnacks zerosnacks self-assigned this Jul 11, 2024
@zerosnacks
Copy link
Member

zerosnacks commented Jul 11, 2024

@DaniPopes / @mattsse Is this something we still would want to do?

Related file: https://github.com/foundry-rs/foundry/blob/master/crates/common/src/ens.rs

@zerosnacks zerosnacks added this to the v1.0.0 milestone Jul 26, 2024
@zerosnacks
Copy link
Member

zerosnacks commented Jul 30, 2024

Related ticket in Alloy: alloy-rs/alloy#491, leaving this ticket open to mirror the one in Alloy so it can be moved over once implemented.

@zerosnacks zerosnacks removed their assignment Jul 30, 2024
@zerosnacks zerosnacks added the T-blocked Type: blocked label Jul 30, 2024
@zerosnacks zerosnacks removed the T-blocked Type: blocked label Sep 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-internals Area: internals D-chore Difficulty: chore T-debt Type: code debt
Projects
Status: Todo
Development

No branches or pull requests

2 participants