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

Add deprecation notice #3

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Add deprecation notice #3

wants to merge 1 commit into from

Conversation

gndelia
Copy link

@gndelia gndelia commented Feb 21, 2025

This package is no longer useful and should be archived. Viem offers this very same utility.

image

@gndelia gndelia requested a review from gabmontes February 21, 2025 19:28
@gndelia gndelia self-assigned this Feb 21, 2025
Copy link
Member

@gabmontes gabmontes left a comment

Choose a reason for hiding this comment

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

I've been thinking again about this. This package is not deprecated as it is still useful. Not to us but others may find it useful for sure. The truth is that we are not going to maintain this repo, but there is no maintenance to do as the ERC-20 ABI will no change and there are no dependencies to update either.

Perhaps we could add a simpler note advising viem users to use the ABI exposed by them instead of this one. But that's it.

@mbonfoster mbonfoster removed their request for review February 25, 2025 19:53
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