Skip to content

Check addresses to be checksummed #938

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

Open
JosepBove opened this issue Mar 13, 2025 · 0 comments
Open

Check addresses to be checksummed #938

JosepBove opened this issue Mar 13, 2025 · 0 comments
Labels
A-ci Area: CI tooling K-fix Kind: fix M-good-first-issue Meta: Good for newcomers

Comments

@JosepBove
Copy link
Member

JosepBove commented Mar 13, 2025

Some addresses in standard-version-sepolia.toml do not have the correct Ethereum checksum format. This causes issues when importing the file in Solidity.

All Ethereum addresses should be correctly checksummed to prevent Solidity import errors, we should implement a check in the CI pipeline to detect incorrectly formatted addresses when modifying addresses.

@JosepBove JosepBove added M-good-first-issue Meta: Good for newcomers K-fix Kind: fix A-ci Area: CI tooling labels Mar 13, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-ci Area: CI tooling K-fix Kind: fix M-good-first-issue Meta: Good for newcomers
Projects
None yet
Development

No branches or pull requests

1 participant