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

fix: Changing ERC20 references to ERC-20 #943

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

Conversation

mzkrasner
Copy link

This PR addresses this issue: #940

  • Changes mentions of ERC20 across documentation and function descriptions to ERC-20
  • Leaves all internal links using erc20 untouched

Copy link

vercel bot commented Jan 18, 2025

@mzkrasner is attempting to deploy a commit to the Consensys Team on Vercel.

A member of the Team first needs to authorize it.

@mzkrasner
Copy link
Author

@jlwllmr

@jlwllmr
Copy link
Collaborator

jlwllmr commented Jan 28, 2025

@mzkrasner thanks, I'm just following up internally as editing the smart contracts reference files may not be appropriate, since they're autogenerated. The usage there is pulled from comments in the smart contracts themselves, so trying to figure out the best approach.

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