wevm / viem

TypeScript Interface for Ethereum
https://viem.sh
Other
2.51k stars 797 forks source link

fix: rename zkSync to ZKsync #2546

Closed JackHamer09 closed 2 months ago

JackHamer09 commented 2 months ago

This creates a minor breaking change because of changed variable and type names.


PR-Codex overview

This PR focuses on renaming zkSync to ZKsync, updating variable names, types, and removing zkSyncTestnet.

Detailed summary

The following files were skipped due to too many changes: site/pages/zksync/actions/signTransaction.md, src/zksync/actions/getTestnetPaymasterAddress.test.ts, src/README.md, src/zksync/types/fee.ts, src/zksync/actions/estimateFee.test.ts, src/zksync/actions/getAllBalances.test.ts, src/zksync/decorators/publicL2.test.ts, src/zksync/utils/assertEip712Request.ts, src/zksync/actions/estimateGasL1ToL2.test.ts, test/src/zksync.ts, src/zksync/actions/getL1ChainId.ts, src/actions/public/verifyHash.test.ts, src/zksync/actions/getL1BatchNumber.ts, test/src/anvil.ts, src/zksync/actions/getLogProof.ts, src/zksync/actions/getBaseTokenL1Address.ts, src/zksync/actions/getMainContractAddress.ts, src/chains/definitions/zkSyncLocalNode.ts, src/chains/definitions/zkSyncInMemoryNode.ts, src/zksync/actions/getBridgehubContractAddress.ts, src/zksync/actions/getTestnetPaymasterAddress.ts, src/zksync/actions/getAllBalances.ts, src/zksync/actions/getL1BatchBlockRange.ts, src/zksync/utils/assertEip712Transaction.ts, src/zksync/actions/getDefaultBridgeAddresses.ts, src/zksync/actions/estimateGasL1ToL2.ts, src/zksync/actions/signEip712Transaction.test.ts, src/zksync/actions/deployContract.test.ts, src/chains/definitions/zkSync.ts, site/pages/zksync/actions/getMainContractAddress.md, src/zksync/actions/sendEip712Transaction.ts, src/zksync/types/chain.ts, src/zksync/actions/sendTransaction.ts, src/zksync/actions/getBlockDetails.ts, src/zksync/actions/signTransaction.ts, src/zksync/actions/getL1BatchDetails.ts, src/zksync/actions/signEip712Transaction.ts, src/zksync/actions/signTransaction.test.ts, site/pages/zksync/actions/getBlockDetails.md, src/zksync/actions/getTransactionDetails.ts, site/pages/zksync/chains.md, src/zksync/types/eip712.ts, site/pages/zksync/actions/getDefaultBridgeAddress.md, src/zksync/actions/estimateFee.ts, src/chains/definitions/zkSyncSepoliaTestnet.ts, src/zksync/utils/getEip712Domain.ts, src/zksync/actions/getRawBlockTransactions.ts, site/pages/zksync/client.md, src/zksync/actions/sendTransaction.test.ts, src/zksync/types/log.ts, src/zksync/serializers.ts, src/zksync/utils/getEip712Domain.test.ts, src/zksync/actions/sendEip712Transaction.test.ts, src/actions/public/getTransactionReceipt.test.ts, src/zksync/types/eip1193.ts, site/pages/docs/chains/zksync.md, src/zksync/index.ts, src/zksync/types/block.ts, src/zksync/formatters.test.ts, site/pages/zksync.mdx, src/zksync/decorators/eip712.test.ts, src/chains/index.ts, src/zksync/decorators/eip712.ts, src/zksync/formatters.ts, src/zksync/formatters.test-d.ts, src/zksync/serializers.test.ts, src/zksync/decorators/publicL2.ts, src/zksync/types/transaction.ts

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

changeset-bot[bot] commented 2 months ago

🦋 Changeset detected

Latest commit: bf799889cd5dce6ede01e290f6436d2e5d73d7d2

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package | Name | Type | | ---- | ----- | | viem | Minor |

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

vercel[bot] commented 2 months ago

@JackHamer09 is attempting to deploy a commit to the Wevm Team on Vercel.

A member of the Team first needs to authorize it.

JackHamer09 commented 2 months ago

Not sure why test fail... Something about chain files but these files are there...

JackHamer09 commented 2 months ago

@tmm added all old imports back with @deprecated jsdoc

tmm commented 2 months ago

Wasn't able to push. Closing in favor of https://github.com/wevm/viem/pull/2586