Skip to content

docs: checksum all v4 deployment addresses - #1157

Open
kongtaoxing wants to merge 2 commits into
Uniswap:mainfrom
kongtaoxing:fix/base-sepolia-checksum-addresses
Open

docs: checksum all v4 deployment addresses#1157
kongtaoxing wants to merge 2 commits into
Uniswap:mainfrom
kongtaoxing:fix/base-sepolia-checksum-addresses

Conversation

@kongtaoxing

Copy link
Copy Markdown

Summary

Normalizes all deployment addresses on the Uniswap v4 deployments page to their EIP-55 checksummed representations.

The update covers all 24 network sections, including mainnets and testnets. Displayed addresses and their corresponding block explorer URLs now use consistent checksum casing.

Only letter casing was changed. The underlying 20-byte address values and contract mappings remain unchanged.

Page:
https://developers.uniswap.org/docs/protocols/v4/deployments

Official deployment data:
https://developers.uniswap.org/deployments.json

EIP-55 specification:
https://eips.ethereum.org/EIPS/eip-55

Type of change

  • Fix (typo, broken link, incorrect or outdated content)
  • New content (guide, page, code example)
  • Update to existing content
  • Other

How has this been verified?

  • Scanned all 212 displayed deployment addresses and their corresponding explorer URLs.
  • Verified that every address occurrence now matches its EIP-55 checksum encoding.
  • Confirmed that lowercasing the updated file produces content identical to the original, ensuring no address values or unrelated content were changed.
  • Ran the Git diff whitespace check successfully.

Applicable screenshots

Not applicable. This change only normalizes address casing.

Anything else reviewers should know?

No contracts, deployments, or network mappings were changed.

@github-actions

github-actions Bot commented Sep 9, 2026

Copy link
Copy Markdown

Thanks for contributing to Uniswap Docs! 🦄

Quick heads up on how this repo works: the docs you see on developers.uniswap.org are built from a separate source repo. When we accept a change here, we port it over there, so your fix might show up on the live site before it lands in this repo.

Updates to this repo come in batches on a regular cadence. When yours is included, you'll be credited as a co-author on the commit, so the contribution counts on your GitHub profile and in Uniswap/docs.

We review every PR but can't merge everything. Either way, someone from the docs team will follow up here once we've taken a look.

One legal bit: by submitting a PR you agree your contribution is licensed under the repo's MIT license.

@github-actions

github-actions Bot commented Sep 9, 2026

Copy link
Copy Markdown

Review summary

This PR normalizes all 24 network sections of content/protocols/v4/deployments.mdx to EIP-55 checksummed address casing (both in the displayed 0x... values and in the corresponding explorer URLs).

Verification performed:

  • Diffed the full PR and extracted every 0x[0-9a-fA-F]{40} occurrence from removed vs. added lines, lowercased both sets, sorted, and hashed them — the two multisets are byte-for-byte identical (same hash). This confirms only letter casing changed; no underlying 20-byte address value, contract mapping, chain ID, network name, or explorer domain was altered anywhere in the file.
  • Additions (146) and deletions (146) are symmetric, consistent with a pure re-casing of existing lines — no lines added/removed/reordered.
  • Spot-checked table structure/formatting: since casing changes don't affect string length, markdown table alignment is preserved, and no frontmatter or surrounding prose was touched.

Caveat for reviewers: my sandboxed review environment didn't have code-execution/network access available, so I wasn't able to independently recompute the Keccak-256 EIP-55 checksum for each of the ~150 changed addresses to confirm the new casing is the mathematically correct checksum (as opposed to just "some different casing"). Given the value-integrity check above passed, worst case here is cosmetic (a wallet/tool that strictly validates checksums could flag a mismatch), not a wrong-address risk. Recommend a quick automated pass (e.g. ethers.getAddress()/viem.getAddress() equality check per address) before merge if that's easy to run in CI, otherwise this looks safe to merge as-is.

No broken links, typos, or spam/content concerns found.

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.

1 participant